[PATCH] D23243: [clang-tidy] enhance modernize-use-bool-literals to check ternary operator
Kirill Bobyrev via cfe-commits
cfe-commits at lists.llvm.org
Sun Aug 7 12:56:06 PDT 2016
omtcyfz added a comment.
In https://reviews.llvm.org/D23243#508097, @aaron.ballman wrote:
> LGTM! We may want to consider BinaryConditionalOperator as well, but that can be in as a separate patch.
Well, I personally have never seen `BinaryConditionalOperator `in my life.
And honestly I'm not a fan of supporting GNU extensions - what makes sense to me is to use `BinaryConditionalOperator` in a check which suggests not using GNU extensions.
Anyway, I'd be glad if @alexfh could jump in and take a look.
https://reviews.llvm.org/D23243
More information about the cfe-commits
mailing list