Removed InfBlocks from SA_FIELD_SELF_ASSIGNMENT

[shai-almog]

No description provided.

[github-actions]

✅ Continuous Quality Report

Test & Coverage

• ✅ Tests: 2281 total, 0 failed, 0 skipped
• 📊 Line coverage: 52.88% [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: 499 findings (Normal: 318, Low: 181) [HTML preview] [Download]
  • Top findings
    • Normal: CodenameOne/src/com/codename1/ads/InnerActive.java:1 – Class doesn't override equals in superclass (rule: EQ_DOESNT_OVERRIDE_EQUALS)
    • Normal: CodenameOne/src/com/codename1/ads/InnerActive.java:149 – Method concatenates strings using + in a loop (rule: SBSC_USE_STRINGBUFFER_CONCATENATION)
    • Normal: CodenameOne/src/com/codename1/capture/Capture.java:262 – Naked notify (rule: NN_NAKED_NOTIFY)
    • Normal: CodenameOne/src/com/codename1/capture/VideoCaptureConstraints.java:193 – equals method fails for subtypes (rule: EQ_GETCLASS_AND_CLASS_CONSTANT)
    • Normal: CodenameOne/src/com/codename1/capture/VideoCaptureConstraints.java:110 – equals() method does not check for null argument (rule: NP_EQUALS_SHOULD_HANDLE_NULL_ARGUMENT)
    • …and 494 more
• ❌ PMD: 2713 findings (P2: 1, P3: 631, P4: 2081) [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:7451 – These nested if statements could be combined (rule: CollapsibleIfStatements)
    • P3: CodenameOne/src/com/codename1/impl/CodenameOneImplementation.java:8471 – These nested if statements could be combined (rule: CollapsibleIfStatements)
    • …and 2708 more
• ❌ Checkstyle: 220209 findings (Warning: 220209) [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 220204 more

Generated automatically by the PR CI workflow.