[PATCH] D29726: [Clang-tidy] Fix for bug 31838: readability-delete-null-pointer does not work for class members

Mads Ravn via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Feb 9 10:03:26 PST 2017


madsravn marked 3 inline comments as done.
madsravn added inline comments.


================
Comment at: clang-tidy/readability/DeleteNullPointerCheck.cpp:46
+                 DeleteExpr, DeleteMemberExpr,
+                 compoundStmt(anyOf(has(DeleteExpr), has(DeleteMemberExpr)),
+                              statementCountIs(1))
----------------
alexfh wrote:
> nit: Will `has(anyOf(DeleteExpr, DeleteMemberExpr))` work?
I could swear that I had tried that. It seems like the obvious way to do it. It worked, though.


https://reviews.llvm.org/D29726





More information about the cfe-commits mailing list