[PATCH] D21298: [Clang-tidy] delete null check

Piotr Padlewski via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Dec 14 15:07:08 PST 2016


Prazek added inline comments.


================
Comment at: clang-tidy/readability/DeleteNullPointerCheck.cpp:33
+  Finder->addMatcher(
+      ifStmt(allOf(hasCondition(
+                       anyOf(PointerCondition, BinaryPointerCheckCondition)),
----------------
I think allOf matcher is redundant here because ifStmt takes variadic number of arguments and matches only if all of them matches.


https://reviews.llvm.org/D21298





More information about the cfe-commits mailing list