Skip to content

GROOVY-12242: instanceof pattern variable scope is not aligned with Java flow scoping (JEP 394) - #2773

Merged
paulk-asert merged 7 commits into
masterfrom
GROOVY-12242
Aug 12, 2026
Merged

GROOVY-12242: instanceof pattern variable scope is not aligned with Java flow scoping (JEP 394)#2773
paulk-asert merged 7 commits into
masterfrom
GROOVY-12242

GROOVY-12242: emit one LocalVariableTable entry per pattern-variable …

208ee07
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded Aug 12, 2026 in 2m 27s

Annotations

Check warning on line 810 in src/main/java/org/codehaus/groovy/classgen/VariableScopeVisitor.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

This block of commented-out lines of code should be removed.

See more on https://sonarcloud.io/project/issues?id=apache_groovy&issues=AZ_sfOqP2dMl_8X6nAJy&open=AZ_sfOqP2dMl_8X6nAJy&pullRequest=2773

Check failure on line 1652 in src/main/java/org/codehaus/groovy/classgen/AsmClassGenerator.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this method to reduce its Cognitive Complexity from 16 to the 15 allowed.

See more on https://sonarcloud.io/project/issues?id=apache_groovy&issues=AZ_nziroIdg5QZyCFXHd&open=AZ_nziroIdg5QZyCFXHd&pullRequest=2773

Check warning on line 3235 in src/main/java/org/apache/groovy/parser/antlr4/AstBuilder.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

This block of commented-out lines of code should be removed.

See more on https://sonarcloud.io/project/issues?id=apache_groovy&issues=AZ_sfOt-2dMl_8X6nAJz&open=AZ_sfOt-2dMl_8X6nAJz&pullRequest=2773

Check warning on line 1261 in src/main/java/org/codehaus/groovy/classgen/VariableScopeVisitor.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this instanceof check and cast with 'instanceof BooleanExpression booleanexpression'

See more on https://sonarcloud.io/project/issues?id=apache_groovy&issues=AZ_lMVyLP2IEEuXp7I0N&open=AZ_lMVyLP2IEEuXp7I0N&pullRequest=2773

Check warning on line 480 in src/main/java/org/codehaus/groovy/classgen/asm/CompileStack.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Declare "endLabels" on a separate line.

See more on https://sonarcloud.io/project/issues?id=apache_groovy&issues=AZ_ze7NBjIxcKW8Oww8z&open=AZ_ze7NBjIxcKW8Oww8z&pullRequest=2773