Use GCC's -Wlogical-op if possible

Gerald Pfeifer gerald at pfeifer.com
Sun Sep 19 06:55:13 CDT 2010


On Mon, 4 Jan 2010, Henri Verbeet wrote:
>> I dont see a reason to take that warning serious, as its not a problem 
>> in any case. in kernel32 it just  happens because we use a Macro. 
>> Taking that case(v==1) out of the Macro leads to harder readable code. 
>> So IMHO i would not make -Wlogical-op the default.
> Not using that macro at all would be even more readable. I'm probably
> missing something obvious, but it seems to just do a wsprintfW() with
> "#%d" as format.

I see this has been comitted via b45d4aa161fbbb905fa9142d2757ff3f7952566d.
Thanks!  Looks like this warning is useful after all. ;-)

In fact, doing a build with -Wlogical-op right now it turns out that with
the change above and the work last year there aren't any open issues on
that front any more and I'll submit a patch to make this used by default.

Gerald


More information about the wine-devel mailing list