On 09/12/2016 02:00 PM, Bernd Edlinger wrote:
Yes, I will update the patch accordingly.
I agree, a statement like "if (x ? 0 : 2)" can be called suspicious as
well, even if the result of x is not ignored as in "if (x ? 1 : 2)".
After I posted this patch, I extended the patch to cover also
suspicious int shift-left operations, at
https://gcc.gnu.org/ml/gcc-patches/2016-09/msg00155.html
[PATCHv2] Add a warning for suspicious use of conditional expressions in
boolean context
Therefore, I would like to rename the warning to -Wint-in-bool-context
so that the other patch can extend that instead of creating even more
warning names with much longer names.
Is renaming the warning OK as well, or could you have a look at the
follow-up patch please?
Renaming is OK as well.
jeff
