diff --git a/pmd-core/src/main/java/net/sourceforge/pmd/RuleSetFactoryCompatibility.java b/pmd-core/src/main/java/net/sourceforge/pmd/RuleSetFactoryCompatibility.java index ffbdc1d4b4..af39be8cf4 100644 --- a/pmd-core/src/main/java/net/sourceforge/pmd/RuleSetFactoryCompatibility.java +++ b/pmd-core/src/main/java/net/sourceforge/pmd/RuleSetFactoryCompatibility.java @@ -70,7 +70,7 @@ public class RuleSetFactoryCompatibility { // PMD 6.0.0 addFilterRuleMoved("java", "controversial", "unnecessary", "UnnecessaryParentheses"); - addFilterRuleMoved("java", "unnecessary", "UnnecessaryParentheses", "UselessParentheses"); + addFilterRuleRenamed("java", "unnecessary", "UnnecessaryParentheses", "UselessParentheses"); } diff --git a/pmd-java/src/test/resources/net/sourceforge/pmd/lang/java/rule/unnecessary/xml/UselessParentheses.xml b/pmd-java/src/test/resources/net/sourceforge/pmd/lang/java/rule/unnecessary/xml/UselessParentheses.xml index 8117737399..bde1e112fa 100644 --- a/pmd-java/src/test/resources/net/sourceforge/pmd/lang/java/rule/unnecessary/xml/UselessParentheses.xml +++ b/pmd-java/src/test/resources/net/sourceforge/pmd/lang/java/rule/unnecessary/xml/UselessParentheses.xml @@ -371,18 +371,6 @@ public class Useless { ]]> - - From controversial/UnnecessaryParentheses: Typecast - 0 - - - From controversial/UnnecessaryParentheses: #537 Trivial case with additive expression 1 @@ -445,6 +433,28 @@ public class Useless { ]]> + + From controversial/UnnecessaryParentheses: bad, returning variable in parens + 1 + + + + From controversial/UnnecessaryParentheses: Failure case, returning literal + 1 + +