[clang-tools-extra] r259640 - clang-tidy: [misc-unused-parameters] Ignore template instantiations.
    Daniel Jasper via cfe-commits 
    cfe-commits at lists.llvm.org
       
    Wed Feb  3 03:33:19 PST 2016
    
    
  
Author: djasper
Date: Wed Feb  3 05:33:18 2016
New Revision: 259640
URL: http://llvm.org/viewvc/llvm-project?rev=259640&view=rev
Log:
clang-tidy: [misc-unused-parameters] Ignore template instantiations.
No functional changes intended as we should already do the
corresponding fixes when visiting the primary template. There are
existing tests that verify that we do change unused parameters of
templated functions.
Modified:
    clang-tools-extra/trunk/clang-tidy/misc/UnusedParametersCheck.cpp
Modified: clang-tools-extra/trunk/clang-tidy/misc/UnusedParametersCheck.cpp
URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/clang-tidy/misc/UnusedParametersCheck.cpp?rev=259640&r1=259639&r2=259640&view=diff
==============================================================================
--- clang-tools-extra/trunk/clang-tidy/misc/UnusedParametersCheck.cpp (original)
+++ clang-tools-extra/trunk/clang-tidy/misc/UnusedParametersCheck.cpp Wed Feb  3 05:33:18 2016
@@ -104,7 +104,8 @@ void UnusedParametersCheck::warnOnUnused
 void UnusedParametersCheck::check(const MatchFinder::MatchResult &Result) {
   const auto *Function = Result.Nodes.getNodeAs<FunctionDecl>("function");
   if (!Function->doesThisDeclarationHaveABody() ||
-      !Function->hasWrittenPrototype())
+      !Function->hasWrittenPrototype() ||
+      Function->isTemplateInstantiation())
     return;
   if (const auto *Method = dyn_cast<CXXMethodDecl>(Function))
     if (Method->isLambdaStaticInvoker())
    
    
More information about the cfe-commits
mailing list