hokein accepted this revision.
hokein added a reviewer: hokein.
hokein added a comment.

It looks good to me, but you'd better wait for the approval from @aaron.ballman.

Comment at: clang-tidy/readability/DeleteNullPointerCheck.cpp:55
+      "'if' statement is unnecessary; deleting null pointer has no effect");
+  if (!IfWithDelete->getElse()) {
+    std::string ReplacementText = Lexer::getSourceText(
I would use an early return `if (IfWithDelete->getElse()) return` here.


cfe-commits mailing list

Reply via email to