BAS_BLOATED_ASSIGNMENT_SCOPE: strange violations that hard to understand

[romani]

detected checkstyle/checkstyle#15725 (comment)
Bumps com.mebigfatguy.sb-contrib:sb-contrib from 7.6.4 to 7.6.5.

[ERROR] Medium: Method com.puppycrawl.tools.checkstyle.checks.coding
.UnusedCatchParameterShouldBeUnnamedCheck.isCatchParameterIdentifierCandidate(DetailAST)
 assigns a variable in a larger scope than is needed 
[com.puppycrawl.tools.checkstyle.checks.coding.UnusedCatchParameterShouldBeUnnamedCheck] 
At UnusedCatchParameterShouldBeUnnamedCheck.java:[line 164]
 BAS_BLOATED_ASSIGNMENT_SCOPE

[ERROR] Medium: Method com.puppycrawl.tools.checkstyle.checks.coding
.UnusedCatchParameterShouldBeUnnamedCheck.isCatchParameterIdentifierCandidate(DetailAST) 
assigns a variable in a larger scope than is needed 
[com.puppycrawl.tools.checkstyle.checks.coding.UnusedCatchParameterShouldBeUnnamedCheck] 
At UnusedCatchParameterShouldBeUnnamedCheck.java:[line 166]
 BAS_BLOATED_ASSIGNMENT_SCOPE

[ERROR] Medium: Method com.puppycrawl.tools.checkstyle.checks.imports
.PkgImportControl.matchesAtFrontNoRegex(String) 
assigns a variable in a larger scope than is needed
 [com.puppycrawl.tools.checkstyle.checks.imports.PkgImportControl] At PkgImportControl.java:[line 245] 
BAS_BLOATED_ASSIGNMENT_SCOPE

looks like false positives

[mebigfatguy]

it's a relatively cheesy report, but in the first case

UnusedCatchParameterShouldBeUnnamedCheck.java:[line 164]

for example

you could do at 162

if (isCatchParameterDeclaration) {
return false;
}

and then line 168 becomes

return (hasValidParentToken || isMethodInvocation);

which avoids the execution of some code if isCatchParameterDeclaration is true.