diff options
Diffstat (limited to 'clang-tools-extra/clang-tidy/misc/UnusedParametersCheck.cpp')
| -rw-r--r-- | clang-tools-extra/clang-tidy/misc/UnusedParametersCheck.cpp | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/clang-tools-extra/clang-tidy/misc/UnusedParametersCheck.cpp b/clang-tools-extra/clang-tidy/misc/UnusedParametersCheck.cpp index 02cb6216e38..18e39609f0e 100644 --- a/clang-tools-extra/clang-tidy/misc/UnusedParametersCheck.cpp +++ b/clang-tools-extra/clang-tidy/misc/UnusedParametersCheck.cpp @@ -70,10 +70,11 @@ void UnusedParametersCheck::warnOnUnusedParameter( }; // Comment out parameter name for non-local functions. - if ((Function->isExternallyVisible() && - Function->getStorageClass() != StorageClass::SC_Static) || - UsedByRef()) { + if (Function->isExternallyVisible() || UsedByRef()) { 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 + // clean this up afterwards. MyDiag << FixItHint::CreateReplacement( RemovalRange, (Twine(" /*") + Param->getName() + "*/").str()); return; |
