codeflash-ai/codeflash
1
0
Fork 0
Code Issues 18 Pull requests 33 Projects Releases Packages Wiki Activity Actions 2.4k

fix: correct Java string syntax in line profiler code generation

Browse source 
Fixed two critical bugs in JavaLineProfiler that prevented compilation:

1. OUTPUT_FILE declaration: Changed from repr() (single quotes) to regular
   string interpolation (double quotes). Java requires double quotes for
   string literals.

2. JSON generation: Changed all literal newlines (\n) to escaped newlines
   (\\n) in StringBuilder append calls. Java does not support multi-line
   string literals without escape sequences.

Both bugs caused Java compilation errors. After fixes:
- Code compiles successfully with Maven
- E2E tests pass (instrumentation → compilation → execution → profiling)
- All unit tests pass (8/9, 1 skipped)
- Profile data correctly captured and parsed

Tested with real Java code (Fibonacci.sumFibonacci) - verified hotspot
identification works correctly.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
This commit is contained in:
Mohamed Ashraf 2026-02-10 16:32:29 +00:00
parent 8f67258f7c
commit bbd987ba5a
1 changed files with 9 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

18
codeflash/languages/java/line_profiler.py
View file

@ -125,7 +125,7 @@ class {self.profiler_class} {{
private static final ThreadLocal<Long> lastLineTime = new ThreadLocal<>();
private static final ThreadLocal<String> lastKey = new ThreadLocal<>();
private static final java.util.concurrent.atomic.AtomicInteger totalHits = new java.util.concurrent.atomic.AtomicInteger(0);
private static final String OUTPUT_FILE = {str(self.output_file)!r};
private static final String OUTPUT_FILE = "{str(self.output_file)}";
static class LineStats {{
public final java.util.concurrent.atomic.AtomicLong hits = new java.util.concurrent.atomic.AtomicLong(0);
@ -201,11 +201,11 @@ class {self.profiler_class} {{
// Build JSON with stats
StringBuilder json = new StringBuilder();
json.append("{{\n");
json.append("{{\\n");
boolean first = true;
for (java.util.Map.Entry<String, LineStats> entry : stats.entrySet()) {{
if (!first) json.append(",\n");
if (!first) json.append(",\\n");
first = false;
String key = entry.getKey();
@ -215,16 +215,16 @@ class {self.profiler_class} {{
// Escape quotes in content
content = content.replace("\\"", "\\\\\\"");
json.append(" \\"").append(key).append("\\": {{\n");
json.append(" \\"hits\\": ").append(st.hits.get()).append(",\n");
json.append(" \\"time\\": ").append(st.timeNs.get()).append(",\n");
json.append(" \\"file\\": \\"").append(st.file).append("\\",\n");
json.append(" \\"line\\": ").append(st.line).append(",\n");
json.append(" \\"").append(key).append("\\": {{\\n");
json.append(" \\"hits\\": ").append(st.hits.get()).append(",\\n");
json.append(" \\"time\\": ").append(st.timeNs.get()).append(",\\n");
json.append(" \\"file\\": \\"").append(st.file).append("\\",\\n");
json.append(" \\"line\\": ").append(st.line).append(",\\n");
json.append(" \\"content\\": \\"").append(content).append("\\"\\n");
json.append(" }}");
}}
json.append("\n}}");
json.append("\\n}}");
java.nio.file.Files.write(
outputFile.toPath(),