================
@@ -207,6 +207,13 @@ void ConstCorrectnessCheck::check(const
MatchFinder::MatchResult &Result) {
assert(Variable && LocalScope && Function);
+ // If a variable (e.g. function parameter) is unnamed, don't report it. Being
+ // unnamed already guarantees that the variable can't be accessed, so
+ // 'const'ness (can't be modified) doesn't add extra information. Also, the
+ // messages would be awkward in this case.
+ if (Variable->getDeclName().isIdentifier() && Variable->getName().empty())
----------------
NagyDonat wrote:
Done in
https://github.com/llvm/llvm-project/pull/184388/commits/66e9c0ca2cfdfbd2853cf06bcc1a2c058736e5ef
https://github.com/llvm/llvm-project/pull/184388
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
