Comments resolved.

Author: dmpolukhin

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

@@ -125,7 +125,7 @@ void NamedParameterCheck::check(const MatchFinder::MatchResult &Result) {
       // The fix depends on the InsertPlainNamesInForwardDecls option,
       // whether this is a forward declaration and whether the parameter has
       // a real name.
-      bool IsForwardDeclaration = (!Definition || Function != Definition);
+      const bool IsForwardDeclaration = (!Definition || Function != Definition);
       if (InsertPlainNamesInForwardDecls && IsForwardDeclaration &&
           NewName != FallbackName) {
         // For forward declarations with InsertPlainNamesInForwardDecls enabled,

clang-tools-extra/docs/clang-tidy/checks/readability/named-parameter.rst

@@ -29,6 +29,6 @@ Options
 
 .. option:: InsertPlainNamesInForwardDecls
 
-   When `true`, the check will insert parameter names without comments for
-   forward declarations only. When `false` (default), parameter names are
-   always inserted as comments (e.g., ``/*param*/``).
+   If set to `true`, the check will insert parameter names without comments for
+   forward declarations only. Otherwise, the check will insert parameter names
+   as comments (e.g., ``/*param*/``). Default is `false`.