A RetroSearch Logo

Home - News ( United States | United Kingdom | Italy | Germany ) - Football scores

Search Query:

Showing content from https://github.com/pmd/pmd/issues/4518 below:

false-positive with multiple for-loop indices · Issue #4518 · pmd/pmd · GitHub

Affects PMD Version: 6.x

Rule: UnusedLocalVariable

Description:

Index variables of for loops should be ignored by this rule. This works if a single index is used, but for multiple variables, a false positive violation is reported.

Note: This is already fixed with PMD 7.0.0-rc1.
Found via #3123.

Code Sample demonstrating the issue:

public class UnusedLocalVarsMultiple {
    public void sample1() {
        int a = 0, b = 1; // a and b unused (line 3)
    }
    public void sample2() {
        int a = 0, b = a; // only b unused (line 6)
    }
    public void sample3() {
        for (int i = 0; i < 10; i++); // i is loop index, is used and should be ignored
    }
    public void sample4() {
        for (int i = 0, j = 0; i < 10; i++, j++); // i and j are loop indices, both are used (line 12)
    }
    public void sample5() {
        for (int i = 0, j = 0; i < 10; i++); // i and j are loop indices, but j is not used (line 15)
    }
}

Expected outcome:

PMD reports a violation at line 12, but that's wrong. That's a false positive.


RetroSearch is an open source project built by @garambo | Open a GitHub Issue

Search and Browse the WWW like it's 1997 | Search results from DuckDuckGo

HTML: 3.2 | Encoding: UTF-8 | Version: 0.7.4