alexfh accepted this revision. alexfh added a comment. This revision is now accepted and ready to land.
Looks good! I'll fix the last issue (see the comment below) and commit the patch for you. ================ Comment at: clang-tidy/modernize/UseNullptrCheck.cpp:80 @@ +79,3 @@ + bool NeedsSpace = + isAlphanumeric(*FullSourceLoc(PreviousLocation, SM).getCharacterData()); + Check.diag(Range.getBegin(), "use nullptr") << FixItHint::CreateReplacement( ---------------- `FullSourceLoc` makes things more complicated here, `SM.getCharacterData(PreviousLocation)` would be better and shorter. http://reviews.llvm.org/D12081 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
