Re: add -Wlogical-op to standard compiler options?

On Thu, Nov 15, 2012 at 1:46 PM, Peter Eisentraut <peter_e@gmx.net> wrote:
> On 11/15/12 9:40 AM, Tom Lane wrote:
>> Peter Eisentraut <peter_e@gmx.net> writes:
>>> I think it might be worth adding -Wlogical-op to the standard warning
>>> options (for supported compilers, determined by configure test).
>>
>> Does that add any new warnings with the current source code, and if
>> so what?
>
> none

Using gcc (GCC) 4.4.6 20120305 (Red Hat 4.4.6-4),  I get dozens of
warnings, all apparently coming from somewhere in the MemSet macro.

example:

pl_handler.c:301: warning: logical '&&' with non-zero constant will
always evaluate as true

Probably has something to do with:
                       /* \                        *      If MEMSET_LOOP_LIMIT == 0, optimizer
should find \                        *      the whole "if" false at compile time. \                        */ \
             MEMSET_LOOP_LIMIT != 0) \
 

Cheers,

Jeff