* ObjC patch - do not replace token->value with canonical spelling
@ 2010-09-22 9:26 Nicola Pero
2010-09-28 22:29 ` Nicola Pero
2010-09-29 11:11 ` Mike Stump
0 siblings, 2 replies; 3+ messages in thread
From: Nicola Pero @ 2010-09-22 9:26 UTC (permalink / raw)
To: gcc-patches
This one-liner patch removes a difference between the Objective-C
and C/C++/Objective-C++ parsers (it's the first of a set of patches
- this tiny difference prevents me from moving code around in more
interesting patches later, so I want to get it out of the way first) ;-)
No regressions. Ok to apply ?
Thanks
Index: gcc/ChangeLog
===================================================================
--- gcc/ChangeLog (revision 164498)
+++ gcc/ChangeLog (working copy)
@@ -1,3 +1,9 @@
+2010-09-21 Nicola Pero <nicola.pero@meta-innovation.com>
+
+ * c-parser.c (c_lex_one_token): In Objective-C, do not replace
+ token->value with the canonical spelling. Do exactly like C and
+ C++ and leave it as it is.
+
2010-09-21 Anatoly Sokolov <aesok@post.ru>
* config/rs6000/rs6000.h (OUTPUT_ADDR_CONST_EXTRA): Remove macros.
Index: gcc/c-parser.c
===================================================================
--- gcc/c-parser.c (revision 164498)
+++ gcc/c-parser.c (working copy)
@@ -242,8 +242,6 @@ c_lex_one_token (c_parser *parser, c_token *token)
&& (!OBJC_IS_PQ_KEYWORD (rid_code)
|| parser->objc_pq_context))
{
- /* Return the canonical spelling for this keyword. */
- token->value = ridpointers[(int) rid_code];
token->type = CPP_KEYWORD;
token->keyword = rid_code;
break;
^ permalink raw reply [flat|nested] 3+ messages in thread
* RE: ObjC patch - do not replace token->value with canonical spelling
2010-09-22 9:26 ObjC patch - do not replace token->value with canonical spelling Nicola Pero
@ 2010-09-28 22:29 ` Nicola Pero
2010-09-29 11:11 ` Mike Stump
1 sibling, 0 replies; 3+ messages in thread
From: Nicola Pero @ 2010-09-28 22:29 UTC (permalink / raw)
To: Mike Stump; +Cc: gcc-patches
ping :-)
Thanks
-----Original Message-----
From: "Nicola Pero" <nicola.pero@meta-innovation.com>
Sent: Wednesday, 22 September, 2010 00:31
To: gcc-patches@gnu.org
Subject: ObjC patch - do not replace token->value with canonical spelling
This one-liner patch removes a difference between the Objective-C
and C/C++/Objective-C++ parsers (it's the first of a set of patches
- this tiny difference prevents me from moving code around in more
interesting patches later, so I want to get it out of the way first) ;-)
No regressions. Ok to apply ?
Thanks
Index: gcc/ChangeLog
===================================================================
--- gcc/ChangeLog (revision 164498)
+++ gcc/ChangeLog (working copy)
@@ -1,3 +1,9 @@
+2010-09-21 Nicola Pero <nicola.pero@meta-innovation.com>
+
+ * c-parser.c (c_lex_one_token): In Objective-C, do not replace
+ token->value with the canonical spelling. Do exactly like C and
+ C++ and leave it as it is.
+
2010-09-21 Anatoly Sokolov <aesok@post.ru>
* config/rs6000/rs6000.h (OUTPUT_ADDR_CONST_EXTRA): Remove macros.
Index: gcc/c-parser.c
===================================================================
--- gcc/c-parser.c (revision 164498)
+++ gcc/c-parser.c (working copy)
@@ -242,8 +242,6 @@ c_lex_one_token (c_parser *parser, c_token *token)
&& (!OBJC_IS_PQ_KEYWORD (rid_code)
|| parser->objc_pq_context))
{
- /* Return the canonical spelling for this keyword. */
- token->value = ridpointers[(int) rid_code];
token->type = CPP_KEYWORD;
token->keyword = rid_code;
break;
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: ObjC patch - do not replace token->value with canonical spelling
2010-09-22 9:26 ObjC patch - do not replace token->value with canonical spelling Nicola Pero
2010-09-28 22:29 ` Nicola Pero
@ 2010-09-29 11:11 ` Mike Stump
1 sibling, 0 replies; 3+ messages in thread
From: Mike Stump @ 2010-09-29 11:11 UTC (permalink / raw)
To: Nicola Pero; +Cc: gcc-patches
On Sep 21, 2010, at 3:31 PM, Nicola Pero wrote:
> This one-liner patch removes a difference between the Objective-C
> and C/C++/Objective-C++ parsers (it's the first of a set of patches
> - this tiny difference prevents me from moving code around in more
> interesting patches later, so I want to get it out of the way first) ;-)
>
> No regressions. Ok to apply ?
Ok. I know what happened, saw c-parser.c and left it to the c people... :-)
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2010-09-28 19:45 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-09-22 9:26 ObjC patch - do not replace token->value with canonical spelling Nicola Pero
2010-09-28 22:29 ` Nicola Pero
2010-09-29 11:11 ` Mike Stump
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).