Re: vasnprintf.c needs IF_LINT, too

2008-05-21 Thread Bruno Haible
Jim Meyering wrote: > Oops. I mistakenly pushed the latter unreviewed change (to avoid > this warning) along with the typo-fixing one. Sorry, Bruno. So here's my review afterwards. - The patch was incomplete: When I turn on all possible NEED_* options in config.h, I get more warnings:

Re: vasnprintf.c needs IF_LINT, too

2008-05-21 Thread Jim Meyering
Jim Meyering <[EMAIL PROTECTED]> wrote: > + avoid a warning from gcc > + * lib/vasnprintf.c (IF_LINT): Define. > + (scale10_round_decimal_long_double): > + Use it to avoid a "may be used uninitialized" warning. Same problem a few lines below: (I'll commit this in a few hours if no

vasnprintf.c needs IF_LINT, too

2008-05-21 Thread Jim Meyering
Oops. I mistakenly pushed the latter unreviewed change (to avoid this warning) along with the typo-fixing one. Sorry, Bruno. cc1: warnings being treated as errors vasnprintf.c: In function 'scale10_round_decimal_long_double': vasnprintf.c:1208: error: 'e' may be used uninitialized in this