token_loc, too. And mention it in the function
> comment.
>
> OK with that change.
Thanks. For the record, this is the updated patch.
From: Dodji Seketeli
Date: Sat, 4 Dec 2010 18:35:47 +0100
Subject: [PATCH 6/7] Kill pedantic warnings on system headers macros
This patch leverag
On 09/16/2011 04:46 AM, Dodji Seketeli wrote:
struct c_declspecs *
-finish_declspecs (struct c_declspecs *specs)
+finish_declspecs (struct c_declspecs *specs,
+ location_t where)
Let's call this first_token_loc, too. And mention it in the function
comment.
OK with that chan
Jason Merrill writes:
> On 07/16/2011 10:37 AM, Dodji Seketeli wrote:
> > + location_t here = c_parser_peek_token (parser)->location;
>
> Perhaps "first_token_loc"?
OK, changed.
>
> > + SYNTAX_ERROR2_AT (prev_virtual_location,
> > + "missing binary opera
On 07/16/2011 10:37 AM, Dodji Seketeli wrote:
+ location_t here = c_parser_peek_token (parser)->location;
Perhaps "first_token_loc"? It's unfortunate that we don't retain the
locations of the individual declspecs, but I don't expect you to fix that.
+ SYNTAX_ERROR2_AT (prev_virt
This patch leverages the virtual location infrastructure to avoid
emitting pedantic warnings related to macros defined in system headers
but expanded in normal TUs.
The point is to make diagnostic routines use virtual locations of
tokens instead of their spelling locations. The diagnostic routines
