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 2013/11/19 01:35:12 UTC

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

Author: ggregory
Date: Tue Nov 19 00:35:11 2013
New Revision: 1543245

URL: http://svn.apache.org/r1543245
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=1543245&r1=1543244&r2=1543245&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 Tue Nov 19 00:35:11 2013
@@ -782,9 +782,8 @@ abstract class AbstractPatriciaTrie<K, V
     public K firstKey() {
         if (size() == 0) {
             throw new NoSuchElementException();
-        } else {
-            return firstEntry().getKey();
         }
+        return firstEntry().getKey();
     }
 
     public K lastKey() {