aaron.ballman accepted this revision.
aaron.ballman added a comment.
LGTM, with one nit. You should wait for @alexfh to sign off before committing
though, since he requested changes.
================
Comment at: clang-tidy/readability/DeleteNullPointerCheck.cpp:53
+ "'if' statement is unnecessary; deleting null pointer has no effect");
+ if (IfWithDelete->getElse()) {
+ return;
----------------
Elide the braces.
https://reviews.llvm.org/D21298
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
