[PATCH] D16922: [clang-tidy] Added check-fixes for misc-virtual-near-miss.

Cong Liu via cfe-commits cfe-commits at lists.llvm.org
Fri Feb 5 09:58:15 PST 2016


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





More information about the cfe-commits mailing list