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

Kirill Bobyrev via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 8 07:55:44 PDT 2016


omtcyfz added inline comments.

================
Comment at: clang-tidy/modernize/UseBoolLiteralsCheck.cpp:41
@@ +40,3 @@
+          // using anyOf(BINDLITERAL(), anything()).
+          anyOf(hasTrueExpression(ignoringParenImpCasts(
+                    integerLiteral().bind("trueBranchLiteral"))),
----------------
aaron.ballman wrote:
> alexfh wrote:
> > I think, Aaron was talking about `eachOf` matcher that will actually generate a separate match for each of its submatchers.
> I *knew* we had something that would do this! Thank you for the reminder, Alex. :-)
Yup. Seems like I learned about something, too :)


https://reviews.llvm.org/D23243





More information about the cfe-commits mailing list