[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