aaron.ballman added a subscriber: aaron.ballman.
aaron.ballman added a comment.
Thank you for working on this! A question below, but also, the patch is missing
test cases for the change.
================
Comment at: ClangTidy.cpp:144
@@ -138,1 +143,3 @@
+
+ Diag << FixItHint::CreateReplacement(Range, Fix.getReplacementText());
++TotalFixes;
----------------
Is there a purpose to emitting the CreateReplacement if the fix isn't
applicable?
http://reviews.llvm.org/D18262
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
