[PATCH] D17958: [clang-tidy] Make 'modernize-use-nullptr' check ignores NULL marcos used in other macros.

Benjamin Kramer via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 11 01:57:42 PST 2016


bkramer accepted this revision.
bkramer added a comment.
This revision is now accepted and ready to land.

Patch looks good, only one cosmetic comment.


================
Comment at: clang-tidy/modernize/UseNullptrCheck.cpp:232
@@ +231,3 @@
+      if (!getMacroAndArgLocations(StartLoc, ImmediateMarcoArgLoc, MacroLoc)
+          || ImmediateMarcoArgLoc != FileLocStart)
+        return skipSubTree();
----------------
This doesn't look clang-formatted.


Repository:
  rL LLVM

http://reviews.llvm.org/D17958





More information about the cfe-commits mailing list