You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cxf.apache.org by ni...@apache.org on 2006/12/21 09:15:24 UTC

svn commit: r489284 - in /incubator/cxf/trunk: api/src/main/java/org/apache/cxf/service/model/OperationInfo.java buildtools/src/main/resources/cxf-pmd-ruleset.xml

Author: ningjiang
Date: Thu Dec 21 00:15:24 2006
New Revision: 489284

URL: http://svn.apache.org/viewvc?view=rev&rev=489284
Log:
Changed the PMD rules back

Modified:
    incubator/cxf/trunk/api/src/main/java/org/apache/cxf/service/model/OperationInfo.java
    incubator/cxf/trunk/buildtools/src/main/resources/cxf-pmd-ruleset.xml

Modified: incubator/cxf/trunk/api/src/main/java/org/apache/cxf/service/model/OperationInfo.java
URL: http://svn.apache.org/viewvc/incubator/cxf/trunk/api/src/main/java/org/apache/cxf/service/model/OperationInfo.java?view=diff&rev=489284&r1=489283&r2=489284
==============================================================================
--- incubator/cxf/trunk/api/src/main/java/org/apache/cxf/service/model/OperationInfo.java (original)
+++ incubator/cxf/trunk/api/src/main/java/org/apache/cxf/service/model/OperationInfo.java Thu Dec 21 00:15:24 2006
@@ -69,8 +69,7 @@
      */
     public final void setName(QName name) {
         if (name == null) {
-            throw new NullPointerException(
-                new Message("OPERATION.NAME.NOT.NULL", LOG).toString());
+            throw new NullPointerException("Operation Name cannot be null.");                
         }        
         opName = name;
     }

Modified: incubator/cxf/trunk/buildtools/src/main/resources/cxf-pmd-ruleset.xml
URL: http://svn.apache.org/viewvc/incubator/cxf/trunk/buildtools/src/main/resources/cxf-pmd-ruleset.xml?view=diff&rev=489284&r1=489283&r2=489284
==============================================================================
--- incubator/cxf/trunk/buildtools/src/main/resources/cxf-pmd-ruleset.xml (original)
+++ incubator/cxf/trunk/buildtools/src/main/resources/cxf-pmd-ruleset.xml Thu Dec 21 00:15:24 2006
@@ -87,7 +87,7 @@
 <!--<rule ref="rulesets/design.xml/CloseConnection"/>-->
 <!--<rule ref="rulesets/design.xml/CompareObjectsWithEquals"/>-->
 <!--<rule ref="rulesets/design.xml/ConfusingTernary"/>-->
-<!--rule ref="rulesets/design.xml/ConstructorCallsOverridableMethod"/-->
+<rule ref="rulesets/design.xml/ConstructorCallsOverridableMethod"/>
 <!--<rule ref="rulesets/design.xml/DefaultLabelNotLastInSwitchStmt"/>-->
 <!--<rule ref="rulesets/design.xml/FinalFieldCouldBeStatic"/>-->
 <rule ref="rulesets/design.xml/IdempotentOperations"/>