#1464 [java] UnnecessaryFinalModifier: false positive on a @SafeVarargs method

[Alexey Yudichev]

It fails here. Removal of final modifier from a @SafeVarargs method breaks compilation, so this rule should ignore @SafeVarargs methods.

public final class InboxContents<T> {
    @SafeVarargs
    public final InboxContents<T> conflateWith(T... values) { // false positive
        return conflateWith(ImmutableList.copyOf(values));
    }
}

[Andreas Dangel]

This bug will be fixed with PMD 5.3.7 and later.

Commit: https://github.com/pmd/pmd/commit/573186be6900170948dba8fcbb0c520f5fd81aab

[Alexey Yudichev]

I can see two 5.3 and 5.4 versions of PDM in the maven central. Will it be fixed in 5.4.2 too?

[Andreas Dangel]

Yes, the fix will be in all the next releases (5.3.7, 5.4.2 and 5.5.0). However, I don't know yet, when I'll release them.
If you need it right now, you can take the SNAPSHOT versions from the files download here:

• https://sourceforge.net/projects/pmd/files/pmd/5.3.7-SNAPSHOT/
• https://sourceforge.net/projects/pmd/files/pmd/5.4.2-SNAPSHOT/