clang-tidy: [misc-unused-parameters] Ignore template instantiations.
authorDaniel Jasper <djasper@google.com>
Wed, 3 Feb 2016 11:33:18 +0000 (11:33 +0000)
committerDaniel Jasper <djasper@google.com>
Wed, 3 Feb 2016 11:33:18 +0000 (11:33 +0000)
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.

llvm-svn: 259640

clang-tools-extra/clang-tidy/misc/UnusedParametersCheck.cpp

index 8be8767..1269c3b 100644 (file)
@@ -104,7 +104,8 @@ void UnusedParametersCheck::warnOnUnusedParameter(
 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())