Skip to content

GROOVY-12023: add a PIC with configurable size for indy in fron of the existing cache #2549

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
blackdrag wants to merge 19 commits into
base: master
Choose a base branch
from

GROOVY-12023: Move call-site flags into CacheableCallSite and fix spr…

da154cf
Select commit
Loading
Failed to load commit list.
Draft

GROOVY-12023: add a PIC with configurable size for indy in fron of the existing cache #2549

GROOVY-12023: Move call-site flags into CacheableCallSite and fix spr…
da154cf
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded May 27, 2026 in 2m 11s

Quality Gate passed

Issues
16 New issues
0 Accepted issues

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

See analysis details on SonarQube Cloud

Annotations

Check warning on line 751 in src/main/java/org/codehaus/groovy/vmplugin/v8/Selector.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use the built-in formatting to construct this argument. 

See more on https://sonarcloud.io/project/issues?id=apache_groovy&issues=AZ5UOjm3i-BUK6nuG57t&open=AZ5UOjm3i-BUK6nuG57t&pullRequest=2549

Check warning on line 161 in src/main/java/org/codehaus/groovy/vmplugin/v8/Selector.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Make skipSwitchPoint a static final constant or non-public and provide accessors if needed. 

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

Check failure on line 362 in src/main/java/org/codehaus/groovy/vmplugin/v8/IndyInterface.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

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

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

Check warning on line 362 in src/main/java/org/codehaus/groovy/vmplugin/v8/IndyInterface.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace generic exceptions with specific library exceptions or a custom exception. 

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

Check warning on line 165 in src/main/java/org/codehaus/groovy/vmplugin/v8/Selector.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Make fallback a static final constant or non-public and provide accessors if needed. 

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

Check warning on line 1207 in src/main/java/org/codehaus/groovy/vmplugin/v8/Selector.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Move this method into "MethodSelector". 

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

Check warning on line 1288 in src/main/java/org/codehaus/groovy/vmplugin/v8/Selector.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Move this method into "MethodSelector". 

See more on https://sonarcloud.io/project/issues?id=apache_groovy&issues=AZ5UOjm4i-BUK6nuG57z&open=AZ5UOjm4i-BUK6nuG57z&pullRequest=2549

Check failure on line 821 in src/main/java/org/codehaus/groovy/vmplugin/v8/Selector.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Define a constant instead of duplicating this literal "invokeMethod" 3 times. 

See more on https://sonarcloud.io/project/issues?id=apache_groovy&issues=AZ5UOjm4i-BUK6nuG57u&open=AZ5UOjm4i-BUK6nuG57u&pullRequest=2549

Check warning on line 440 in src/main/java/org/codehaus/groovy/vmplugin/v8/IndyInterface.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace generic exceptions with specific library exceptions or a custom exception. 

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

Check warning on line 145 in src/main/java/org/codehaus/groovy/vmplugin/v8/Selector.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Make safeNavOnNull a static final constant or non-public and provide accessors if needed. 

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

Check warning on line 734 in src/main/java/org/codehaus/groovy/vmplugin/v8/Selector.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 74 to 64, Complexity from 32 to 14, Nesting Level from 4 to 2, Number of Variables from 10 to 6. 

See more on https://sonarcloud.io/project/issues?id=apache_groovy&issues=AZ5UOjm4i-BUK6nuG570&open=AZ5UOjm4i-BUK6nuG570&pullRequest=2549

Check warning on line 1278 in src/main/java/org/codehaus/groovy/vmplugin/v8/Selector.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Move this method into "MethodSelector". 

See more on https://sonarcloud.io/project/issues?id=apache_groovy&issues=AZ5UOjm4i-BUK6nuG57y&open=AZ5UOjm4i-BUK6nuG57y&pullRequest=2549

Check warning on line 67 in src/main/java/org/codehaus/groovy/vmplugin/v8/IndyInterface.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Declare "THIS_CALL" and all following declarations on a separate line. 

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

Check warning on line 1298 in src/main/java/org/codehaus/groovy/vmplugin/v8/Selector.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Move this method into "MethodSelector". 

See more on https://sonarcloud.io/project/issues?id=apache_groovy&issues=AZ5UOjm4i-BUK6nuG57x&open=AZ5UOjm4i-BUK6nuG57x&pullRequest=2549

Check warning on line 446 in src/main/java/org/codehaus/groovy/vmplugin/v8/IndyInterface.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Merge this if statement with the enclosing one. 

See more on https://sonarcloud.io/project/issues?id=apache_groovy&issues=AZ5JkRRL7b0f9vdM1ub-&open=AZ5JkRRL7b0f9vdM1ub-&pullRequest=2549

Check warning on line 329 in src/main/java/org/codehaus/groovy/tools/DgmConverter.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=AZ5UOjq7i-BUK6nuG577&open=AZ5UOjq7i-BUK6nuG577&pullRequest=2549
View more details on SonarQubeCloud