More comments resolved.

Author: dmpolukhin

clang-tools-extra/clang-tidy/readability/NamedParameterCheck.cpp

@@ -95,7 +95,7 @@ void NamedParameterCheck::check(const MatchFinder::MatchResult &Result) {
 
     for (auto P : UnnamedParams) {
       // Fallback to an unused marker.
-      constexpr StringRef FallbackName = "unused";
+      static constexpr StringRef FallbackName = "unused";
       StringRef NewName = FallbackName;
 
       // If the method is overridden, try to copy the name from the base method
@@ -129,11 +129,10 @@ void NamedParameterCheck::check(const MatchFinder::MatchResult &Result) {
       if (InsertPlainNamesInForwardDecls && IsForwardDeclaration &&
           NewName != FallbackName) {
         // For forward declarations with InsertPlainNamesInForwardDecls enabled,
-        // insert the parameter name without comments
+        // insert the parameter name without comments.
         D << FixItHint::CreateInsertion(Parm->getLocation(),
                                         " " + NewName.str());
       } else {
-        // Default behavior: insert the parameter name as a comment
         D << FixItHint::CreateInsertion(Parm->getLocation(),
                                         " /*" + NewName.str() + "*/");
       }

clang-tools-extra/test/clang-tidy/checkers/readability/named-parameter.cpp

@@ -1,5 +1,6 @@
 // RUN: %check_clang_tidy %s readability-named-parameter %t
-// RUN: %check_clang_tidy -check-suffix=PLAIN-NAMES %s readability-named-parameter %t -- -config="{CheckOptions: [{key: readability-named-parameter.InsertPlainNamesInForwardDecls, value: true}]}"
+// RUN: %check_clang_tidy -check-suffix=PLAIN-NAMES %s readability-named-parameter %t -- \
+// RUN:   -config="{CheckOptions: [{key: readability-named-parameter.InsertPlainNamesInForwardDecls, value: true}]}"
 
 void Method(char *) { /* */ }
 // CHECK-MESSAGES: :[[@LINE-1]]:19: warning: all parameters should be named in a function