it in the function
comment.
OK with that change.
Thanks. For the record, this is the updated patch.
From: Dodji Seketeli do...@redhat.com
Date: Sat, 4 Dec 2010 18:35:47 +0100
Subject: [PATCH 6/7] Kill pedantic warnings on system headers macros
This patch leverages the virtual location
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
Jason Merrill ja...@redhat.com 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
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
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