You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cxf.apache.org by dk...@apache.org on 2017/02/23 19:05:11 UTC

cxf-build-utils git commit: Update PMD rulesets to work without warnings with newer PMD versions

Repository: cxf-build-utils
Updated Branches:
  refs/heads/master d7836448c -> 94a777d42


Update PMD rulesets to work without warnings with newer PMD versions


Project: http://git-wip-us.apache.org/repos/asf/cxf-build-utils/repo
Commit: http://git-wip-us.apache.org/repos/asf/cxf-build-utils/commit/94a777d4
Tree: http://git-wip-us.apache.org/repos/asf/cxf-build-utils/tree/94a777d4
Diff: http://git-wip-us.apache.org/repos/asf/cxf-build-utils/diff/94a777d4

Branch: refs/heads/master
Commit: 94a777d42acc49c9dd0eceea407b5c55a9d34023
Parents: d783644
Author: Daniel Kulp <dk...@apache.org>
Authored: Thu Feb 23 14:04:50 2017 -0500
Committer: Daniel Kulp <dk...@apache.org>
Committed: Thu Feb 23 14:04:50 2017 -0500

----------------------------------------------------------------------
 .../resources/cxf-pmd-ruleset-generated.xml     | 20 ++++++++++----------
 .../src/main/resources/cxf-pmd-ruleset.xml      | 18 +++++++++---------
 2 files changed, 19 insertions(+), 19 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cxf-build-utils/blob/94a777d4/buildtools/src/main/resources/cxf-pmd-ruleset-generated.xml
----------------------------------------------------------------------
diff --git a/buildtools/src/main/resources/cxf-pmd-ruleset-generated.xml b/buildtools/src/main/resources/cxf-pmd-ruleset-generated.xml
index 0967dad..06dfd42 100644
--- a/buildtools/src/main/resources/cxf-pmd-ruleset-generated.xml
+++ b/buildtools/src/main/resources/cxf-pmd-ruleset-generated.xml
@@ -43,15 +43,15 @@
     <rule ref="rulesets/java/basic.xml/BooleanInstantiation"/>
     <rule ref="rulesets/java/basic.xml/CollapsibleIfStatements"/>
     <rule ref="rulesets/java/basic.xml/DoubleCheckedLocking"/>
-    <!--<rule ref="rulesets/java/basic.xml/EmptyCatchBlock"/>-->
-    <!--<rule ref="rulesets/java/basic.xml/EmptyFinallyBlock"/>-->
-    <!--<rule ref="rulesets/java/basic.xml/EmptyIfStmt"/>-->
+    <!--<rule ref="rulesets/java/empty.xml/EmptyCatchBlock"/>-->
+    <!--<rule ref="rulesets/java/empty.xml/EmptyFinallyBlock"/>-->
+    <!--<rule ref="rulesets/java/empty.xml/EmptyIfStmt"/>-->
     <rule ref="rulesets/java/empty.xml/EmptyStatementNotInLoop"/>
-    <!--<rule ref="rulesets/java/basic.xml/EmptyStaticInitializer"/>-->
-    <!--<rule ref="rulesets/java/basic.xml/EmptySwitchStatements"/>-->
-    <!--<rule ref="rulesets/java/basic.xml/EmptySynchronizedBlock"/>-->
-    <!--<rule ref="rulesets/java/basic.xml/EmptyTryBlock"/>-->
-    <!--<rule ref="rulesets/java/basic.xml/EmptyWhileStmt"/>-->
+    <!--<rule ref="rulesets/java/empty.xml/EmptyStaticInitializer"/>-->
+    <!--<rule ref="rulesets/java/empty.xml/EmptySwitchStatements"/>-->
+    <!--<rule ref="rulesets/java/empty.xml/EmptySynchronizedBlock"/>-->
+    <!--<rule ref="rulesets/java/empty.xml/EmptyTryBlock"/>-->
+    <!--<rule ref="rulesets/java/empty.xml/EmptyWhileStmt"/>-->
     <rule ref="rulesets/java/basic.xml/ForLoopShouldBeWhileLoop"/>
     <rule ref="rulesets/java/basic.xml/JumbledIncrementer"/>
     <!--<rule ref="rulesets/java/basic.xml/OverrideBothEqualsAndHashcode"/>-->
@@ -60,7 +60,7 @@
     <rule ref="rulesets/java/unnecessary.xml/UnnecessaryConversionTemporary"/>
     <rule ref="rulesets/java/unnecessary.xml/UnnecessaryFinalModifier"/>
     <rule ref="rulesets/java/unnecessary.xml/UnnecessaryReturn"/>
-    <!--<rule ref="rulesets/java/basic.xml/UselessOverridingMethod"/>-->
+    <!--<rule ref="rulesets/java/unnecessary.xml/UselessOverridingMethod"/>-->
 
     <!--<rule ref="rulesets/java/braces.xml/ForLoopsMustUseBraces"/>-->
     <!--<rule ref="rulesets/java/braces.xml/IfElseStmtsMustUseBraces"/>-->
@@ -126,7 +126,7 @@
     <!--rule ref="rulesets/java/design.xml/UnnecessaryLocalBeforeReturn"/-->
     <!--<rule ref="rulesets/java/design.xml/UseLocaleWithCaseConversions"/>-->
     <!--<rule ref="rulesets/java/design.xml/UseNotifyAllInsteadOfNotify"/>-->
-    <!--<rule ref="rulesets/java/design.xml/UseSingleton"/>-->
+    <!--<rule ref="rulesets/java/design.xml/UseUtilityClass"/>-->
 
     <!--<rule ref="rulesets/java/finalizers.xml/EmptyFinalizer"/>-->
     <!--<rule ref="rulesets/java/finalizers.xml/FinalizeOnlyCallsSuperFinalize"/>-->

http://git-wip-us.apache.org/repos/asf/cxf-build-utils/blob/94a777d4/buildtools/src/main/resources/cxf-pmd-ruleset.xml
----------------------------------------------------------------------
diff --git a/buildtools/src/main/resources/cxf-pmd-ruleset.xml b/buildtools/src/main/resources/cxf-pmd-ruleset.xml
index 195ff96..603737c 100644
--- a/buildtools/src/main/resources/cxf-pmd-ruleset.xml
+++ b/buildtools/src/main/resources/cxf-pmd-ruleset.xml
@@ -34,15 +34,15 @@
     <rule ref="rulesets/java/basic.xml/BooleanInstantiation"/>
     <rule ref="rulesets/java/basic.xml/CollapsibleIfStatements"/>
     <rule ref="rulesets/java/basic.xml/DoubleCheckedLocking"/>
-    <!--<rule ref="rulesets/java/basic.xml/EmptyCatchBlock"/>-->
-    <!--<rule ref="rulesets/java/basic.xml/EmptyFinallyBlock"/>-->
-    <!--<rule ref="rulesets/java/basic.xml/EmptyIfStmt"/>-->
+    <!--<rule ref="rulesets/java/empty.xml/EmptyCatchBlock"/>-->
+    <!--<rule ref="rulesets/java/empty.xml/EmptyFinallyBlock"/>-->
+    <!--<rule ref="rulesets/java/empty.xml/EmptyIfStmt"/>-->
     <rule ref="rulesets/java/empty.xml/EmptyStatementNotInLoop"/>
-    <!--<rule ref="rulesets/java/basic.xml/EmptyStaticInitializer"/>-->
-    <!--<rule ref="rulesets/java/basic.xml/EmptySwitchStatements"/>-->
-    <!--<rule ref="rulesets/java/basic.xml/EmptySynchronizedBlock"/>-->
+    <!--<rule ref="rulesets/java/empty.xml/EmptyStaticInitializer"/>-->
+    <!--<rule ref="rulesets/java/empty.xml/EmptySwitchStatements"/>-->
+    <!--<rule ref="rulesets/java/empty.xml/EmptySynchronizedBlock"/>-->
     <!--<rule ref="rulesets/java/java/basic.xml/EmptyTryBlock"/>-->
-    <!--<rule ref="rulesets/java/basic.xml/EmptyWhileStmt"/>-->
+    <!--<rule ref="rulesets/java/empty.xml/EmptyWhileStmt"/>-->
     <rule ref="rulesets/java/basic.xml/ForLoopShouldBeWhileLoop"/>
     <rule ref="rulesets/java/basic.xml/JumbledIncrementer"/>
     <!--<rule ref="rulesets/java/basic.xml/OverrideBothEqualsAndHashcode"/>-->
@@ -51,7 +51,7 @@
     <rule ref="rulesets/java/unnecessary.xml/UnnecessaryConversionTemporary"/>
     <rule ref="rulesets/java/unnecessary.xml/UnnecessaryFinalModifier"/>
     <rule ref="rulesets/java/unnecessary.xml/UnnecessaryReturn"/>
-    <!--<rule ref="rulesets/java/basic.xml/UselessOverridingMethod"/>-->
+    <!--<rule ref="rulesets/java/unnecessary.xml/UselessOverridingMethod"/>-->
 
     <!--<rule ref="rulesets/java/braces.xml/ForLoopsMustUseBraces"/>-->
     <!--<rule ref="rulesets/java/braces.xml/IfElseStmtsMustUseBraces"/>-->
@@ -117,7 +117,7 @@
     <rule ref="rulesets/java/design.xml/UnnecessaryLocalBeforeReturn"/>
     <!--<rule ref="rulesets/java/design.xml/UseLocaleWithCaseConversions"/>-->
     <!--<rule ref="rulesets/java/design.xml/UseNotifyAllInsteadOfNotify"/>-->
-    <!--<rule ref="rulesets/java/design.xml/UseSingleton"/>-->
+    <!--<rule ref="rulesets/java/design.xml/UseUtilityClass"/>-->
 
     <!--<rule ref="rulesets/java/finalizers.xml/EmptyFinalizer"/>-->
     <!--<rule ref="rulesets/java/finalizers.xml/FinalizeOnlyCallsSuperFinalize"/>-->