You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by gg...@apache.org on 2014/10/30 01:23:37 UTC

svn commit: r1635355 - /commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/trie/AbstractPatriciaTrie.java

Author: ggregory
Date: Thu Oct 30 00:23:36 2014
New Revision: 1635355

URL: http://svn.apache.org/r1635355
Log:
Statement unnecessarily nested within else clause.

Modified:
    commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/trie/AbstractPatriciaTrie.java

Modified: commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/trie/AbstractPatriciaTrie.java
URL: http://svn.apache.org/viewvc/commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/trie/AbstractPatriciaTrie.java?rev=1635355&r1=1635354&r2=1635355&view=diff
==============================================================================
--- commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/trie/AbstractPatriciaTrie.java (original)
+++ commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/trie/AbstractPatriciaTrie.java Thu Oct 30 00:23:36 2014
@@ -429,9 +429,8 @@ abstract class AbstractPatriciaTrie<K, V
             if (current.bitIndex <= path.bitIndex) {
                 if (!current.isEmpty() && compareKeys(key, current.key)) {
                     return removeEntry(current);
-                } else {
-                    return null;
                 }
+                return null;
             }
 
             path = current;