Index: clang-tidy/misc/UnusedParametersCheck.cpp =================================================================== --- clang-tidy/misc/UnusedParametersCheck.cpp +++ clang-tidy/misc/UnusedParametersCheck.cpp @@ -16,6 +16,13 @@ namespace clang { namespace tidy { +namespace { +bool isOverrideMethod(const FunctionDecl *Function) { + if (const auto *MD = dyn_cast(Function)) + return MD->size_overridden_methods() > 0 || MD->hasAttr(); + return false; +} +} // namespace void UnusedParametersCheck::registerMatchers(MatchFinder *Finder) { Finder->addMatcher(functionDecl().bind("function"), this); @@ -63,20 +70,15 @@ auto MyDiag = diag(Param->getLocation(), "parameter '%0' is unused") << Param->getName(); - auto UsedByRef = [&] { - return !ast_matchers::match( - decl(hasDescendant( - declRefExpr(to(equalsNode(Function)), - unless(hasAncestor( - callExpr(callee(equalsNode(Function)))))))), - *Result.Context->getTranslationUnitDecl(), *Result.Context) - .empty(); - }; + auto DeclRefExpr = + declRefExpr(to(equalsNode(Function)), + unless(hasAncestor(callExpr(callee(equalsNode(Function)))))); // Comment out parameter name for non-local functions. if (Function->isExternallyVisible() || !Result.SourceManager->isInMainFile(Function->getLocation()) || - UsedByRef()) { + !ast_matchers::match(DeclRefExpr, *Result.Context).empty()|| + isOverrideMethod(Function)) { SourceRange RemovalRange(Param->getLocation(), Param->getLocEnd()); // Note: We always add a space before the '/*' to not accidentally create a // '*/*' for pointer types, which doesn't start a comment. clang-format will Index: test/clang-tidy/misc-unused-parameters.cpp =================================================================== --- test/clang-tidy/misc-unused-parameters.cpp +++ test/clang-tidy/misc-unused-parameters.cpp @@ -127,6 +127,16 @@ useFunction(&C::h); } +class Base { + virtual void f(int i); +}; + +class Derived : public Base { + void f(int i) override {} +// CHECK-MESSAGES: :[[@LINE-1]]:14: warning +// CHECK-FIXES: void f(int /*i*/) override {} +}; + } // end namespace template void someFunctionTemplate(T b, T e) { (void)b; (void)e; }