* Ralf Wildenhues wrote on Tue, Apr 24, 2007 at 08:53:46AM CEST:
> * Charles Wilson wrote on Tue, Apr 24, 2007 at 04:34:41AM CEST:
> > Ralf Wildenhues wrote:
>
> >>> for (i=0; i<argc+1; i++)
> >>> {
> >>> - DEBUG("(main) newargz[%d] : %s\n",i,newargz[i]);
> >>> + LTWRAPPER_DEBUGPRINTF("(main) newargz[%d] : %s\n",i,newargz[i]);
> >>> ;
> >> What's that extra ; for BTW?
> >
> > If !DEBUGWRAPPER, then LTWRAPPER_DEBUGPRINTF() goes away completely, and
> > gcc complains about the empty body in the for loop.
>
> D'oh. Thanks.
No, wrong. If LTWRAPPER_DEBUGPRINTF goes away, then the semicolon at
the end of the LTWRAPPER_DEBUGPRINTF line remains. I don't see how
there can be a warning with one but no warning with two semicolons.
FWIW, my GCC 4.0.3 doesn't warn at all (with -O2 -W -Wall).
Cheers,
Ralf
