Patchwork Patch for objc/23710

login
register
mail settings
Submitter Eric Botcazou
Date Sept. 21, 2010, 10:15 p.m.
Message ID <201009220015.15539.ebotcazou@adacore.com>
Download mbox | patch
Permalink /patch/65385/
State New
Headers show

Comments

Eric Botcazou - Sept. 21, 2010, 10:15 p.m.
> This patch fixes objc/23710 ("objc front-end should not
> "abort" after erroring out about method definition not in
> class context).

Please post ChangeLog in the body of the message.
Nicola Pero - Sept. 21, 2010, 10:28 p.m.
> Please post ChangeLog in the body of the message.

Sure, will do


> c-family/ has its own ChangeLog so the first 2 entries must be moved to there 
> without the prefix.

Ok, I fixed the problem.


Thanks
Eric Botcazou - Sept. 22, 2010, 6:50 a.m.
> Ok, I fixed the problem.

Thanks.  There is another one just below though. :-)
Nicola Pero - Sept. 22, 2010, 7:27 a.m.
>> Ok, I fixed the problem.
>
> Thanks.  There is another one just below though. :-)

Ok ... I addressed that one as well.

Thanks

Patch

Index: gcc/ChangeLog
===================================================================
--- gcc/ChangeLog	(revision 164389)
+++ gcc/ChangeLog	(working copy)
@@ -1,3 +1,14 @@ 
+2010-09-19  Nicola Pero  <nicola.pero@meta-innovation.com>
+
+	PR/objc 23710
+	* c-family/c-common.h (objc_start_method_definition): Return bool
+	instead of void.
+	* c-family/stub-objc.c (objc_start_method_definition): Return bool
+	instead of void.
+	* c-parser.c (c_parser_objc_method_definition): Check the return
+	value of objc_start_method_definition and if false is returned,
+	parse the method definition but emit no code.

c-family/ has its own ChangeLog so the first 2 entries must be moved to there 
without the prefix.