You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by si...@apache.org on 2011/10/21 23:59:53 UTC

svn commit: r1187572 - /commons/proper/digester/trunk/src/main/java/org/apache/commons/digester3/xmlrules/FromXmlRulesModule.java

Author: simonetripodi
Date: Fri Oct 21 21:59:53 2011
New Revision: 1187572

URL: http://svn.apache.org/viewvc?rev=1187572&view=rev
Log:
fixed PMD violation: These nested if statements could be combined

Modified:
    commons/proper/digester/trunk/src/main/java/org/apache/commons/digester3/xmlrules/FromXmlRulesModule.java

Modified: commons/proper/digester/trunk/src/main/java/org/apache/commons/digester3/xmlrules/FromXmlRulesModule.java
URL: http://svn.apache.org/viewvc/commons/proper/digester/trunk/src/main/java/org/apache/commons/digester3/xmlrules/FromXmlRulesModule.java?rev=1187572&r1=1187571&r2=1187572&view=diff
==============================================================================
--- commons/proper/digester/trunk/src/main/java/org/apache/commons/digester3/xmlrules/FromXmlRulesModule.java (original)
+++ commons/proper/digester/trunk/src/main/java/org/apache/commons/digester3/xmlrules/FromXmlRulesModule.java Fri Oct 21 21:59:53 2011
@@ -103,7 +103,7 @@ public abstract class FromXmlRulesModule
     }
 
     /**
-     * 
+     *
      */
     protected abstract void loadRules();
 
@@ -122,12 +122,9 @@ public abstract class FromXmlRulesModule
         this.inputSource.add( inputSource );
 
         String systemId = inputSource.getSystemId();
-        if ( systemId != null )
+        if ( systemId != null && !systemIds.add( systemId ) )
         {
-            if ( !systemIds.add( systemId ) )
-            {
-                addError( "XML rules file '%s' already bound", systemId );
-            }
+            addError( "XML rules file '%s' already bound", systemId );
         }
     }