Exclude PMD UselessParentheses rule in core-unittests

[shai-almog]

Motivation

• Suppress PMD false positives from the UselessParentheses rule in the core-unittests module to avoid noisy static analysis failures.

Description

• Add maven/core-unittests/pmd.xml with an exclusion for UselessParentheses and configure the maven-pmd-plugin in maven/core-unittests/pom.xml to use the new ruleset.

Testing

• No automated tests were run as part of this change.

---

Codex Task

[github-actions]

✅ Continuous Quality Report

Test & Coverage

• ✅ Tests: 2282 total, 0 failed, 0 skipped
• 📊 Line coverage: 52.87% [HTML preview] [Download]
  • Lowest covered classes
    • com.codename1.io.rest.RequestBuilder$FetchAsPropertyListActionListener – 0.00%
    • com.codename1.io.rest.RequestBuilder$FetchAsPropertiesActionListener – 0.00%
    • com.codename1.ui.Display$EdtException – 0.00%
    • com.codename1.ui.BrowserWindow$BrowserForm – 0.00%
    • com.codename1.ui.plaf.CSSBorder$LinearGradient – 0.00%
    • com.codename1.io.rest.RequestBuilder$GetAsStringAsyncImplActionListener – 0.00%
    • com.codename1.io.rest.RequestBuilder$GetAsJsonMapActionListener – 0.00%
    • com.codename1.payment.Purchase$SynchronizeReceiptsSyncRunnable – 0.00%
    • com.codename1.io.rest.RequestBuilder$GetAsBytesAsyncImplActionListener – 0.00%
    • com.codename1.io.rest.RequestBuilder$FetchAsJsonMapActionListener – 0.00%

Static Analysis

• ✅ SpotBugs: 0 findings (no issues) [Report archive]
• ❌ PMD: 6485 findings (P1: 305, P2: 42, P3: 5836, P4: 302) [HTML preview] [Download]
  • Top findings
    • P1: CodenameOne/src/com/codename1/charts/util/MathHelper.java:29 – A class which only has private constructors should be final (rule: ClassWithOnlyPrivateConstructorsShouldBeFinal)
    • P1: CodenameOne/src/com/codename1/components/ClearableTextField.java:42 – A class which only has private constructors should be final (rule: ClassWithOnlyPrivateConstructorsShouldBeFinal)
    • P1: CodenameOne/src/com/codename1/components/FileEncodedImage.java:44 – A class which only has private constructors should be final (rule: ClassWithOnlyPrivateConstructorsShouldBeFinal)
    • P1: CodenameOne/src/com/codename1/components/FileEncodedImage.java:123 – Return an empty collection rather than null. (rule: ReturnEmptyCollectionRatherThanNull)
    • P1: CodenameOne/src/com/codename1/components/FileEncodedImageAsync.java:44 – A class which only has private constructors should be final (rule: ClassWithOnlyPrivateConstructorsShouldBeFinal)
    • …and 6480 more
• ❌ Checkstyle: 220941 findings (Warning: 220941) [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 220936 more

Generated automatically by the PR CI workflow.