Migrate example project and build tooling to Java 17; add compatibility stubs and version parsing

[shai-almog]

Motivation

• Modernize the example multi-module hellocodenameone project and CI/workspace scripts to use Java 17 as the compile and test target.
• Avoid fragile parsing of Java version strings and hard failures when verifying library java versions.
• Provide minimal compatibility stubs for Java string-concat API classes used internally to prevent missing-symbol issues when compiling against newer runtime APIs.

Description

• Updated hellocodenameone pom and module pom files to set <maven.compiler.source> and <maven.compiler.target> to 17, and bumped project java.version and test java versions to 17.
• Added Java 17 detection and preference to wrapper and helper scripts (mvnw, mvnw.cmd, run.*, build.*, build.bat, mvnw helpers) and ensured JAVA_HOME is set to a JDK 17 when available, and updated build-ios-app.sh to run Maven using JAVA17_HOME while logging versions.
• Added a small test Java17LanguageFeaturesTest in the example common test sources to validate Java 17 language features compile in the test classpath.
• Introduced a robust parseJavaVersion helper in CN1BuildMojo and InstallCn1libsMojo to safely parse human-readable Java version strings and avoid NumberFormatException when comparing library/project java versions.
• Added stub implementation files for java.lang.invoke.StringConcatFactory in two locations to provide compatibility placeholders used by string concatenation handling.
• Minor GitHub Actions workflow whitespace cleanup and ensured CocoaPods tooling step remains in the macOS job.

Testing

• Ran the example packaging flow for the iOS example via the build script which invokes ./mvnw package -DskipTests and the script completed successfully.
• Executed mvn test for the hellocodenameone/common test module to verify Java17LanguageFeaturesTest compiles and runs, and the test passed.
• Verified shell and batch script paths and java -version checks by running mvnw/wrapper bootstrap locally and confirming JAVA_HOME selection logic prefers a JDK 17 installation.

---

Codex Task

[github-actions]

✅ Continuous Quality Report

Test & Coverage

• ✅ Tests: 2314 total, 0 failed, 0 skipped
• 📊 Line coverage: 53.15% [HTML preview] [Download]
  • Lowest covered classes
    • com.codename1.ui.Display$EdtException – 0.00%
    • com.codename1.ui.plaf.CSSBorder$LinearGradient – 0.00%
    • com.codename1.util.EasyThread$InQueueRunnable – 0.00%
    • com.codename1.components.FloatingActionButton$ReleaseActionListener – 0.00%
    • com.codename1.io.Oauth2$RefreshTokenActionListener – 0.00%
    • com.codename1.util.EasyThread$RunAndWaitRunnable – 0.00%
    • com.codename1.components.ToastBar$FlushAnimationCallback – 0.00%
    • com.codename1.ui.plaf.CSSBorder$ColorStop – 0.00%
    • com.codename1.ui.ElevationComparator – 0.00%
    • com.codename1.ui.spinner.TimeSpinner3D$MinuteRowFormatter – 0.00%

Static Analysis

• SpotBugs [HTML preview] [Download]
  • ❌ ByteCodeTranslator: 97 findings (High: 2, Normal: 68, Low: 27)
    • Top findings
      • High: com/codename1/tools/translator/BytecodeMethod.java:559 – Method com.codename1.tools.translator.BytecodeMethod.findWritableFields(Set) seems to be useless (rule: UC_USELESS_VOID_METHOD)
      • High: com/codename1/tools/translator/bytecodes/TryCatch.java:75 – Write to static field com.codename1.tools.translator.bytecodes.TryCatch.hasTryCatch from instance method com.codename1.tools.translator.bytecodes.TryCatch.appendInstruction(StringBuilder, List) (rule: ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD)
      • Normal: com/codename1/tools/translator/ByteCodeClass.java:1791 – com.codename1.tools.translator.ByteCodeClass.getBaseClassObject() may expose internal representation by returning ByteCodeClass.baseClassObject (rule: EI_EXPOSE_REP)
      • …and 94 more
  • ❌ android: 412 findings (High: 24, Normal: 196, Low: 192)
    • Top findings
      • High: CodenameOne/src/com/codename1/impl/CodenameOneThread.java:48 – com.codename1.impl.CodenameOneThread.STACK_FRAME_SIZE isn't final but should be (rule: MS_SHOULD_BE_FINAL)
      • High: com/codename1/impl/android/AndroidImplementation.java:1067 – Write to static field com.codename1.impl.android.AndroidImplementation.context from instance method com.codename1.impl.android.AndroidImplementation.setContext(Context) (rule: ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD)
      • High: com/codename1/impl/android/AndroidImplementation.java:1268 – com.codename1.impl.android.AndroidImplementation.isDarkMode() has Boolean return type and returns explicit null (rule: NP_BOOLEAN_RETURN_NULL)
      • …and 409 more
  • ❌ codenameone-maven-plugin: 477 findings (High: 1, Normal: 313, Low: 163)
    • Top findings
      • High: com/codename1/maven/AbstractCN1Mojo.java:807 – com.codename1.maven.AbstractCN1Mojo.OS isn't final but should be (rule: MS_SHOULD_BE_FINAL)
      • Normal: com/codename1/builders/AndroidGradleBuilder.java:1018 – Dead store to googleServicesJson in com.codename1.builders.AndroidGradleBuilder.build(File, BuildRequest) (rule: DLS_DEAD_LOCAL_STORE)
      • Normal: com/codename1/builders/AndroidGradleBuilder.java:1043 – Possible null pointer dereference in com.codename1.builders.AndroidGradleBuilder.build(File, BuildRequest) due to return value of called method (rule: NP_NULL_ON_SOME_PATH_FROM_RETURN_VALUE)
      • …and 474 more
  • ❌ core-unittests: 1 findings (Normal: 1)
    • Top findings
      • Normal: CodenameOne/src/com/codename1/ui/util/MutableResource.java:239 – com.codename1.ui.util.MutableResource.saveL10N(DataOutputStream, Hashtable) makes inefficient use of keySet iterator instead of entrySet iterator (rule: WMI_WRONG_MAP_ITERATOR)
  • ❌ ios: 125 findings (High: 4, Normal: 75, Low: 46)
    • Top findings
      • High: com/codename1/impl/ios/IOSImplementation.java:1223 – com.codename1.impl.ios.IOSImplementation.isDarkMode() has Boolean return type and returns explicit null (rule: NP_BOOLEAN_RETURN_NULL)
      • High: com/codename1/impl/ios/IOSImplementation.java:128 – com.codename1.impl.ios.IOSImplementation.nativeInstance isn't final but should be (rule: MS_SHOULD_BE_FINAL)
      • High: com/codename1/impl/ios/IOSImplementation.java:214 – Write to static field com.codename1.impl.ios.IOSImplementation.instance from instance method com.codename1.impl.ios.IOSImplementation.init(Object) (rule: ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD)
      • …and 122 more
• ✅ PMD: 0 findings (no issues) [Report archive]
• ✅ Checkstyle: 0 findings (no issues) [Report archive]

Generated automatically by the PR CI workflow.

[shai-almog]

Android screenshot updates

Compared 32 screenshots: 31 matched, 1 error.

• graphics-draw-line — comparison error. Comparison error: PNG chunk truncated before CRC while processing: /home/runner/work/_temp/cn1ss-Sv9XIz/graphics-draw-line.png

  No preview available for this screenshot.
  Full-resolution PNG saved as graphics-draw-line.png in workflow artifacts.

Native Android coverage

• 📊 Line coverage: 7.30% (3711/50862 lines covered) [HTML preview] (artifact android-coverage-report, jacocoAndroidReport/html/index.html)
  • Other counters: instruction 5.65% (18059/319659), branch 2.75% (856/31113), complexity 3.40% (1011/29741), method 5.98% (828/13853), class 9.63% (176/1827)
  • Lowest covered classes
    • kotlin.collections.kotlin.collections.ArraysKt___ArraysKt – 0.00% (0/6327 lines covered)
    • kotlin.collections.unsigned.kotlin.collections.unsigned.UArraysKt___UArraysKt – 0.00% (0/2384 lines covered)
    • org.jacoco.agent.rt.internal_b6258fc.asm.org.jacoco.agent.rt.internal_b6258fc.asm.ClassReader – 0.00% (0/1519 lines covered)
    • kotlin.collections.kotlin.collections.CollectionsKt___CollectionsKt – 0.00% (0/1148 lines covered)
    • org.jacoco.agent.rt.internal_b6258fc.asm.org.jacoco.agent.rt.internal_b6258fc.asm.MethodWriter – 0.00% (0/923 lines covered)
    • kotlin.sequences.kotlin.sequences.SequencesKt___SequencesKt – 0.00% (0/712 lines covered)
    • kotlin.text.kotlin.text.StringsKt___StringsKt – 0.00% (0/623 lines covered)
    • org.jacoco.agent.rt.internal_b6258fc.asm.org.jacoco.agent.rt.internal_b6258fc.asm.Frame – 0.00% (0/564 lines covered)
    • kotlin.collections.kotlin.collections.ArraysKt___ArraysJvmKt – 0.00% (0/495 lines covered)
    • kotlinx.coroutines.kotlinx.coroutines.JobSupport – 0.00% (0/423 lines covered)

[github-actions]

✅ ByteCodeTranslator Quality Report

Test & Coverage

• ✅ Tests: 146 total, 0 failed, 2 skipped

Benchmark Results

• Execution Time: 10258 ms

• Hotspots (Top 20 sampled methods):

  • 20.90% java.util.ArrayList.indexOf (375 samples)
  • 18.67% java.lang.String.indexOf (335 samples)
  • 17.34% com.codename1.tools.translator.Parser.isMethodUsed (311 samples)
  • 7.41% com.codename1.tools.translator.BytecodeMethod.addToConstantPool (133 samples)
  • 4.79% java.lang.Object.hashCode (86 samples)
  • 3.73% com.codename1.tools.translator.ByteCodeClass.calcUsedByNative (67 samples)
  • 2.34% java.lang.System.identityHashCode (42 samples)
  • 1.56% java.lang.StringBuilder.append (28 samples)
  • 1.56% com.codename1.tools.translator.Parser.generateClassAndMethodIndexHeader (28 samples)
  • 1.51% com.codename1.tools.translator.BytecodeMethod.optimize (27 samples)
  • 1.23% com.codename1.tools.translator.Parser.cullMethods (22 samples)
  • 1.17% com.codename1.tools.translator.BytecodeMethod.appendMethodC (21 samples)
  • 1.00% java.lang.StringCoding.encode (18 samples)
  • 0.95% com.codename1.tools.translator.ByteCodeClass.markDependent (17 samples)
  • 0.95% java.io.UnixFileSystem.getBooleanAttributes0 (17 samples)
  • 0.89% com.codename1.tools.translator.BytecodeMethod.appendCMethodPrefix (16 samples)
  • 0.89% sun.nio.fs.UnixNativeDispatcher.open0 (16 samples)
  • 0.67% sun.nio.ch.FileDispatcherImpl.write0 (12 samples)
  • 0.56% java.io.FileOutputStream.writeBytes (10 samples)
  • 0.56% com.codename1.tools.translator.BytecodeMethod.isMethodUsedByNative (10 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.