[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