[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.



More information about the Gcc-patches mailing list