[PATCH] D23243: [clang-tidy] enhance modernize-use-bool-literals to check ternary operator

Aaron Ballman via cfe-commits cfe-commits at lists.llvm.org
Sun Aug 7 15:38:47 PDT 2016


On Sun, Aug 7, 2016 at 6:33 PM, Piotr Padlewski
<piotr.padlewski at gmail.com> wrote:
> Prazek added a comment.
>
> Yea, I also have never heard of it. I don't think it is worth even discussing

Just because you've never heard of a compiler extension that gets
pointed out during review does not mean it's not worth discussing. Not
everyone has heard of every compiler vendor extension we support,
that's why we do public review in a wide audience. :-) Again, I am not
suggesting the OP do this as part of this patch.

~Aaron

>
>
> https://reviews.llvm.org/D23243
>
>
>


More information about the cfe-commits mailing list