CORE-57 - Surface javac diagnostics for runtime compile failures #180

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

Open

sam-ross wants to merge 2 commits into develop from feature/CORE-57-surface-javac-diagnostics

Open

CORE-57 - Surface javac diagnostics for runtime compile failures #180

CORE-57 - Adds PR improvements

SonarQubeCloud / SonarCloud Code Analysis succeeded May 18, 2026 in 13s

Quality Gate passed

Issues
3 New issues
0 Accepted issues

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

See analysis details on SonarQube Cloud

Annotations

Check failure on line 308 in src/main/java/net/openhft/compiler/CachedCompiler.java

sonarqubecloud / SonarCloud Code Analysis

Replace this call to "replaceAll()" by a call to the "replace()" method.

See more on https://sonarcloud.io/project/issues?id=OpenHFT_Java-Runtime-Compiler&issues=AZ48DGFCdKoBbZ036FAi&open=AZ48DGFCdKoBbZ036FAi&pullRequest=180

Check warning on line 196 in src/main/java/net/openhft/compiler/CachedCompiler.java

sonarqubecloud / SonarCloud Code Analysis

Replace the synchronized class "StringBuffer" by an unsynchronized one such as "StringBuilder".

See more on https://sonarcloud.io/project/issues?id=OpenHFT_Java-Runtime-Compiler&issues=AZ48DGFCdKoBbZ036FAh&open=AZ48DGFCdKoBbZ036FAh&pullRequest=180

Check warning on line 254 in src/main/java/net/openhft/compiler/CachedCompiler.java

sonarqubecloud / SonarCloud Code Analysis

Replace this "Map.get()" and condition with a call to "Map.computeIfAbsent()".

See more on https://sonarcloud.io/project/issues?id=OpenHFT_Java-Runtime-Compiler&issues=AZ48DGFCdKoBbZ036FAg&open=AZ48DGFCdKoBbZ036FAg&pullRequest=180

View more details on SonarQubeCloud