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 cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits