[PATCH] Warn about bogus conditional operators

Manuel López-Ibáñez lopezibanez@gmail.com
Sat Jan 17 10:27:00 GMT 2009


2009/1/17 Andi Kleen <andi@firstfloor.org>:
>> I'll propose something along the lines of:
>>
>> > +      warning (OPT_Wparentheses,
>> > +              "the omitted middle operand in ?: will always be %<true%>, suggest explicit middle operand");
>
> Makes sense.
>
>>
>> I also think that having a new option for this warning is too much.
>> This can be included in Wparentheses or another option.
>
> Ok i can put it into Wparenthesis
>
> Thanks for the review.
>

I should have pointed out that I am not an official reviewer. I can't
accept/reject your patch. An actual reviewer may actually disagree
with me.

Cheers,

Manuel.



More information about the Gcc-patches mailing list