Re: Patches #219 - #224

[Feico de Boer]

 > ----- Original Message -----
 > From: "Pawel Kot" <address@hidden>
 > To: <address@hidden>
 > Cc: <address@hidden>
 > Sent: Monday, April 08, 2002 10:30 AM
 > Subject: Patches #219 - #224
...

 > #221:
 > Could you please look into win32 directory and make it compatible with
it?

What exactly should I make compatible? Please give a hint with regard to
filenames or whatever.

 > #224:
 > +#    define dump while(0)
 > +#    define dprintf while(0)
 >
 > Are you sure that syntax is correct?

According to me and a co-worker that used to be in compiler design, yes it
is.

What happens:
    dprintf( "some format string: %s %d\n", s, n );
gets translated to:
    while(0);

Which is in essence an zero iteration loop of an empty statement. Your
compiler must be pretty dump if it generates any code for this. Alternatives
are replacing by nothing, semi-colon becomes empty statement or curly-braces
({}) that become an empty basic block.

Regards,

Feico