[PATCH] D21936: [clang-tidy] UnnecessaryValueParamCheck - only warn for virtual methods

[Felix Berger via cfe-commits]

flx added inline comments.

================
Comment at: test/clang-tidy/performance-unnecessary-value-param.cpp:234
@@ +233,3 @@
+};
+
+struct PositiveNonVirualMethod {
----------------
alexfh wrote:
> Please add a test for an override not explicitly marked "virtual" or "override".
That case is already covered in line 180. I moved these tests up now to have them grouped together.


http://reviews.llvm.org/D21936