[PATCH] D16179: [clang-tidy] Handle decayed types and other improvements in VirtualNearMiss check.

[Gábor Horváth via cfe-commits]

xazax.hun marked an inline comment as done.
xazax.hun added a comment.

In http://reviews.llvm.org/D16179#326746, @alexfh wrote:

> > ... What about a configuration option to also report near misses when only a qualifier is missing?
>
>
> Might be a useful thing. We should first check if it makes sense to always ignore a qualifier.


In this iteration of the patch I made the checker ignore the qualifiers. I did not get any results from running it on clang and llvm.


================
Comment at: clang-tidy/misc/VirtualNearMissCheck.cpp:126-127
@@ -127,4 +125,4 @@
   for (unsigned I = 0; I < NumParamA; I++) {
-    if (BaseMD->getParamDecl(I)->getType() !=
-        DerivedMD->getParamDecl(I)->getType())
+    if (getDecayedType(BaseMD->getParamDecl(I)->getType()) !=
+        getDecayedType(DerivedMD->getParamDecl(I)->getType()))
       return false;
----------------
This might not be representative, but I did not get any results on the LLVM codebase.


http://reviews.llvm.org/D16179