Add support for ReadWriteLock in ParparVM

[google-labs-jules]

This PR adds support for ReadWriteLock and ReentrantReadWriteLock to the ParparVM Java API.

Changes:

• Added vm/JavaAPI/src/java/util/concurrent/locks/ReadWriteLock.java
• Added vm/JavaAPI/src/java/util/concurrent/locks/ReentrantReadWriteLock.java
• Added vm/tests/src/test/java/com/codename1/tools/translator/ReadWriteLockIntegrationTest.java covering basic locking, writer exclusion, reentrancy, and correct translation.

The implementation uses standard synchronization primitives available in the ParparVM environment. Serialization support is included with custom readObject logic to handle transient fields.

---

PR created automatically by Jules for task 10497757767453200611 started by @shai-almog

[google-labs-jules]

👋 Jules, reporting for duty! I'm here to lend a hand with this pull request.

When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down.

I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job!

For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with @jules. You can find this option in the Pull Request section of your global Jules UI settings. You can always switch back!

---

For security, I will only act on instructions from the user who triggered this task.

New to Jules? Learn more at jules.google/docs.

[github-actions]

✅ Continuous Quality Report

Test & Coverage

• ✅ Tests: 2280 total, 0 failed, 0 skipped
• 📊 Line coverage: 52.91% [HTML preview] [Download]
  • Lowest covered classes
    • com.codename1.ui.BrowserWindow$BrowserForm – 0.00%
    • com.codename1.processing.TextEvaluator – 3.85%
    • com.codename1.charts.views.PieChart – 4.00%
    • com.codename1.charts.views.PieMapper – 4.88%
    • com.codename1.charts.views.ScatterChart – 5.32%
    • com.codename1.ui.list.GenericListCellRenderer$Monitor – 5.33%
    • com.codename1.ui.Display$EdtException – 5.56%
    • com.codename1.testing.AbstractTest – 6.05%
    • com.codename1.ui.Transform$ImmutableTransform – 6.06%
    • com.codename1.charts.views.DoughnutChart – 6.25%

Static Analysis

• ❌ SpotBugs: 688 findings (High: 56, Normal: 423, Low: 209) [HTML preview] [Download]
  • Top findings
    • High: CodenameOne/src/com/codename1/charts/views/XYChart.java:128 – Call to equals() comparing different types (rule: EC_UNRELATED_TYPES)
    • High: CodenameOne/src/com/codename1/components/ImageViewer.java:310 – Condition has no effect (rule: UC_USELESS_CONDITION)
    • High: CodenameOne/src/com/codename1/components/ToastBar.java:1021 – Potentially ambiguous invocation of either an inherited or outer method (rule: IA_AMBIGUOUS_INVOCATION_OF_INHERITED_OR_OUTER_METHOD)
    • High: CodenameOne/src/com/codename1/facebook/FaceBookAccess.java:191 – Reliance on default encoding (rule: DM_DEFAULT_ENCODING)
    • High: CodenameOne/src/com/codename1/io/BufferedInputStream.java:441 – Reliance on default encoding (rule: DM_DEFAULT_ENCODING)
    • …and 683 more
• ❌ PMD: 2711 findings (P2: 1, P3: 634, P4: 2076) [HTML preview] [Download]
  • Top findings
    • P2: CodenameOne/src/com/codename1/ui/Display.java:1022 – Avoid using a branching statement as the last in a loop. (rule: AvoidBranchingStatementAsLastInLoop)
    • P3: CodenameOne/src/com/codename1/charts/views/LineChart.java:208 – These nested if statements could be combined (rule: CollapsibleIfStatements)
    • P3: CodenameOne/src/com/codename1/facebook/FacebookRESTService.java:76 – Avoid unused constructor parameters such as 'token'. (rule: UnusedFormalParameter)
    • P3: CodenameOne/src/com/codename1/impl/CodenameOneImplementation.java:8454 – These nested if statements could be combined (rule: CollapsibleIfStatements)
    • P3: CodenameOne/src/com/codename1/impl/FullScreenAdService.java:103 – These nested if statements could be combined (rule: CollapsibleIfStatements)
    • …and 2706 more
• ❌ Checkstyle: 220023 findings (Warning: 220023) [HTML preview] [Download]
  • Top findings
    • Warning: CodenameOne/src/com/codename1/db/ThreadSafeDatabase.java:30:1 – Extra separation in import group before 'java.io.IOException' (rule: CustomImportOrderCheck)
    • Warning: CodenameOne/src/com/codename1/db/ThreadSafeDatabase.java:32 – First sentence of Javadoc is missing an ending period. (rule: SummaryJavadocCheck)
    • Warning: CodenameOne/src/com/codename1/db/ThreadSafeDatabase.java:36 – Line is longer than 100 characters (found 143). (rule: LineLengthCheck)
    • Warning: CodenameOne/src/com/codename1/db/ThreadSafeDatabase.java:39:5 – 'member def modifier' has incorrect indentation level 4, expected level should be 2. (rule: IndentationCheck)
    • Warning: CodenameOne/src/com/codename1/db/ThreadSafeDatabase.java:40:5 – 'member def modifier' has incorrect indentation level 4, expected level should be 2. (rule: IndentationCheck)
    • …and 220018 more

Generated automatically by the PR CI workflow.

[shai-almog]

iOS screenshot updates

Compared 30 screenshots: 23 matched, 6 updated, 1 missing reference.

• BrowserComponent — updated screenshot. Screenshot differs (1206x2622 px, bit depth 8).

  
  Preview info: Preview provided by instrumentation.
  Full-resolution PNG saved as BrowserComponent.png in workflow artifacts.

• graphics-draw-arc — updated screenshot. Screenshot differs (1206x2622 px, bit depth 8).

  
  Preview info: JPEG preview quality 20; JPEG preview quality 20; downscaled to 603x1311.
  Full-resolution PNG saved as graphics-draw-arc.png in workflow artifacts.

• graphics-draw-gradient — updated screenshot. Screenshot differs (1206x2622 px, bit depth 8).

  
  Preview info: JPEG preview quality 10; JPEG preview quality 10; downscaled to 844x1835.
  Full-resolution PNG saved as graphics-draw-gradient.png in workflow artifacts.

• graphics-draw-round-rect — missing reference. Reference screenshot missing at /Users/runner/work/CodenameOne/CodenameOne/scripts/ios/screenshots/graphics-draw-round-rect.png.

  
  Preview info: JPEG preview quality 10; JPEG preview quality 10; downscaled to 844x1835.
  Full-resolution PNG saved as graphics-draw-round-rect.png in workflow artifacts.

• graphics-draw-string — updated screenshot. Screenshot differs (1206x2622 px, bit depth 8).

  
  Preview info: JPEG preview quality 10; JPEG preview quality 10; downscaled to 422x918.
  Full-resolution PNG saved as graphics-draw-string.png in workflow artifacts.

• graphics-draw-string-decorated — updated screenshot. Screenshot differs (1206x2622 px, bit depth 8).

  
  Preview info: JPEG preview quality 10; JPEG preview quality 10; downscaled to 603x1311.
  Full-resolution PNG saved as graphics-draw-string-decorated.png in workflow artifacts.

• kotlin — updated screenshot. Screenshot differs (1206x2622 px, bit depth 8).

  
  Preview info: Preview provided by instrumentation.
  Full-resolution PNG saved as kotlin.png in workflow artifacts.

Benchmark Results

• VM Translation Time: 266 seconds
• Compilation Time: 52 seconds

Detailed Performance Metrics

Metric	Duration
Build Time Statistics
Setup & Unzip	24947 ms
Extract Extensions	20 ms
Google Services Setup	2 ms
Scan Classes	449 ms
Extract Libs	925 ms
Inject Build Hints	26 ms
Generate Unit Tests	2 ms
Generate Stubs	908 ms
Compile Stubs	2042 ms
Generate Icons	796 ms
Prepare ParparVM	125 ms
ParparVM Execution	154354 ms
Post-VM Setup	143 ms
CocoaPods	4285 ms
Finalize	40 ms
Total Time	189065 msMaven Overhead : 77000 ms
CocoaPods Install (Script)	1000 ms
Simulator Boot (Run)	41000 ms
App Install	15000 ms
App Launch	9000 ms
Test Execution	137000 ms

[github-actions]

✅ ByteCodeTranslator Quality Report

Test & Coverage

• ✅ Tests: 151 total, 0 failed, 0 skipped

Benchmark Results

• Execution Time: 10132 ms

• Hotspots (Top 20 sampled methods):

  • 22.20% java.lang.String.indexOf (392 samples)
  • 17.50% com.codename1.tools.translator.Parser.isMethodUsed (309 samples)
  • 14.55% com.codename1.tools.translator.Parser.addToConstantPool (257 samples)
  • 10.48% java.util.ArrayList.indexOf (185 samples)
  • 4.87% java.lang.Object.hashCode (86 samples)
  • 3.34% java.lang.System.identityHashCode (59 samples)
  • 3.06% com.codename1.tools.translator.ByteCodeClass.calcUsedByNative (54 samples)
  • 2.15% com.codename1.tools.translator.ByteCodeClass.fillVirtualMethodTable (38 samples)
  • 1.81% com.codename1.tools.translator.BytecodeMethod.optimize (32 samples)
  • 1.30% com.codename1.tools.translator.BytecodeMethod.equals (23 samples)
  • 1.19% java.lang.Integer.getChars (21 samples)
  • 1.19% java.io.FileOutputStream.writeBytes (21 samples)
  • 1.02% com.codename1.tools.translator.Parser.generateClassAndMethodIndexHeader (18 samples)
  • 0.91% java.lang.StringBuilder.append (16 samples)
  • 0.85% java.util.TreeMap.getEntry (15 samples)
  • 0.85% java.io.FileOutputStream.open0 (15 samples)
  • 0.74% java.io.FileInputStream.open0 (13 samples)
  • 0.68% com.codename1.tools.translator.Parser.cullMethods (12 samples)
  • 0.68% com.codename1.tools.translator.BytecodeMethod.appendCMethodPrefix (12 samples)
  • 0.68% com.codename1.tools.translator.ByteCodeClass.markDependent (12 samples)

• ⚠️ Coverage report not generated.

Static Analysis

• ✅ SpotBugs: no findings (report was not generated by the build).
• ⚠️ PMD report not generated.
• ⚠️ Checkstyle report not generated.

Generated automatically by the PR CI workflow.