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/05/10 21:54:53 UTC

svn commit: r1101617 - /commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/SetNestedPropertiesRule.java

Author: simonetripodi
Date: Tue May 10 19:54:53 2011
New Revision: 1101617

URL: http://svn.apache.org/viewvc?rev=1101617&view=rev
Log:
removed unnecessary else statements

Modified:
    commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/SetNestedPropertiesRule.java

Modified: commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/SetNestedPropertiesRule.java
URL: http://svn.apache.org/viewvc/commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/SetNestedPropertiesRule.java?rev=1101617&r1=1101616&r2=1101617&view=diff
==============================================================================
--- commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/SetNestedPropertiesRule.java (original)
+++ commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/SetNestedPropertiesRule.java Tue May 10 19:54:53 2011
@@ -338,21 +338,17 @@ public class SetNestedPropertiesRule ext
                     // constructor.
                     return rules;
                 }
-                else {
-                    // The "real" rules class has rules that match the current
-                    // node, so we return this list *plus* the rule passed to
-                    // this object's constructor.
-                    //
-                    // It might not be safe to modify the returned list,
-                    // so clone it first.
-                    LinkedList<Rule> newMatch = new LinkedList<Rule>(match);
-                    newMatch.addLast(rule);
-                    return newMatch;
-                }
-            }            
-            else {
-                return match;
+                // The "real" rules class has rules that match the current
+                // node, so we return this list *plus* the rule passed to
+                // this object's constructor.
+                //
+                // It might not be safe to modify the returned list,
+                // so clone it first.
+                LinkedList<Rule> newMatch = new LinkedList<Rule>(match);
+                newMatch.addLast(rule);
+                return newMatch;
             }
+            return match;
         }
         
         public List<Rule> rules() {