New C parser [patch]

Zack Weinberg zack@codesourcery.com
Tue Oct 26 01:03:00 GMT 2004


Andrew Pinski <pinskia@physics.uc.edu> writes:

> On Oct 25, 2004, at 8:03 PM, Zack Weinberg wrote:
>
>>> +	  c_parser_error (parser,
>>> +			  "expected ':', ',', ';' or '__attribute__'");
>>> +	  break;
>>
>> These want %<%>ifying.  There are several other cases - I'm not going
>> to call them all out explicitly.
>
> But I will say that we have C++ diagnostic regressions because of
> "%<%>ifying".  The reason why we have this problem is because
> c_parser_error takes a string but only does %s on it rather than passing
> it on to the diagnostic mechanism.

c_parse(r)_error have serious i18n issues already: I don't think this
one more makes much difference.  The way forward is to remove
c_parse_error entirely...

zw



More information about the Gcc-patches mailing list