diff mbox

Patch for objc/23710

Message ID 201009220015.15539.ebotcazou@adacore.com
State New
Headers show

Commit Message

Eric Botcazou Sept. 21, 2010, 10:15 p.m. UTC 
> 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.

Comments

Nicola Pero Sept. 21, 2010, 10:28 p.m. UTC | #1 
> 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. UTC | #2 
> Ok, I fixed the problem.

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

Ok ... I addressed that one as well.

Thanks
diff mbox

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.