Use GCC's -Wlogical-op if possible

[Gerald Pfeifer]

On Sun, 3 Jan 2010, Austin English wrote:
>> On my FreeBSD test system I am see no warnings triggered by -Wlogical-op
>> any more.  How does it look on your side?
> ole32:
> usrmarshal.c:485: warning: logical ?&&? with non-zero constant will
> always evaluate as true
> usrmarshal.c:1098: warning: logical ?&&? with non-zero constant will
> always evaluate as true
> usrmarshal.c:1290: warning: logical ?&&? with non-zero constant will
> always evaluate as true
> 
> oleaut32:
> variant.c:2090: warning: logical ?||? with non-zero constant will
> always evaluate as true
> variant.c:2090: warning: logical ?&&? with non-zero constant will
> always evaluate as true
> 
> comctl32/tests:
> tab.c:520: warning: logical ?&&? with non-zero constant will always
> evaluate as true
> tab.c:540: warning: logical ?&&? with non-zero constant will always
> evaluate as true
> tab.c:563: warning: logical ?&&? with non-zero constant will always
> evaluate as true
> tab.c:978: warning: logical ?&&? with non-zero constant will always
> evaluate as true

I had a patch for this one (comctl32/tests) which I received feedback on 
and need to brush up.  I should be able to do so coming week.  Anybody 
volunteering to look into the other ones?

> kernel32/tests:
> atom.c:70: warning: logical ?||? with non-zero constant will always
> evaluate as true

Gerald