misc gengtype micro-patch (use macro for predefined typedef)

Basile Starynkevitch basile@starynkevitch.net
Sat Nov 6 15:46:00 GMT 2010


On Sat, 6 Nov 2010 15:35:17 +0100
Richard Guenther <richard.guenther@gmail.com> wrote:

>> +#define POS_HERE(Call) do { pos.file = this_file; pos.line = __LINE__; \
>> +       Call;} while(0)
>> +      POS_HERE (do_scalar_typedef ("CUMULATIVE_ARGS", &pos));
>> +      POS_HERE (do_scalar_typedef ("REAL_VALUE_TYPE", &pos));

> This doesn't look like a 1:1 transformation - who increments pos.line?

pos.line is not supposed to be incremented, but to give the accurate
line number.

It was incremented in 4.5 to avoid using __LINE__ repeatedly. (that was
a poor way of doing it, IMHO)

Then, I indented gengtype and the line increment got on seperate line,
which sucks.

Cheers

-- 
Basile STARYNKEVITCH         http://starynkevitch.net/Basile/
email: basile<at>starynkevitch<dot>net mobile: +33 6 8501 2359
8, rue de la Faiencerie, 92340 Bourg La Reine, France
*** opinions {are only mine, sont seulement les miennes} ***



More information about the Gcc-patches mailing list