Skip to content

GROOVY-9192: Implement optional parser error recovery for Parrot parser#2724

Open
daniellansun wants to merge 1 commit into
masterfrom
GROOVY-9192
Open

GROOVY-9192: Implement optional parser error recovery for Parrot parser#2724
daniellansun wants to merge 1 commit into
masterfrom
GROOVY-9192

GROOVY-9192: Implement optional parser error recovery for Parrot parser

1997ba3
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded Jul 19, 2026 in 2m 14s

Quality Gate passed

Issues
7 New issues
0 Accepted issues

Measures
0 Security Hotspots
86.2% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

Annotations

Check warning on line 175 in src/test/java/org/apache/groovy/parser/antlr4/internal/ErrorStrategyTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor the code of the lambda to have only one invocation possibly throwing a runtime exception. 

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

Check warning on line 137 in src/test/java/org/apache/groovy/parser/antlr4/internal/ErrorStrategyTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor the code of the lambda to have only one invocation possibly throwing a runtime exception. 

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

Check warning on line 100 in src/main/java/org/apache/groovy/parser/antlr4/internal/AbstractFriendlyErrorStrategy.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this unused method parameter "recognizer". 

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

Check warning on line 194 in src/test/java/org/apache/groovy/parser/antlr4/internal/ErrorStrategyTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use assertSame instead. 

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

Check failure on line 340 in src/test/java/org/apache/groovy/parser/antlr4/internal/ErrorStrategyTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Add at least one assertion to this test case. 

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

Check warning on line 79 in src/test/java/org/apache/groovy/parser/antlr4/internal/ErrorStrategyTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use assertSame instead. 

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

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

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

A "Brain Method" was detected. Refactor it to reduce at least one of the following metrics: LOC from 70 to 64, Complexity from 17 to 14, Nesting Level from 3 to 2, Number of Variables from 24 to 6. 

See more on https://sonarcloud.io/project/issues?id=apache_groovy&issues=AZ960OVxMvoxo5ITeFwW&open=AZ960OVxMvoxo5ITeFwW&pullRequest=2724
View more details on SonarQubeCloud