congliu added inline comments.

================
Comment at: test/clang-tidy/misc-virtual-near-miss.cpp:1
@@ -1,2 +1,2 @@
 // RUN: %check_clang_tidy %s misc-virtual-near-miss %t
 
----------------
alexfh wrote:
> Please add a test ensuring replacements are applied correctly in templates 
> with multiple instantiations and in macros.
Now the fixes work fine for macros. But for template, the behavior is, as the 
test shows, two warnings were generated, no fix was applied. Is this what we 
want or should we change the strategy?


http://reviews.llvm.org/D16922



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to