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:32:43 UTC

svn commit: r1543244 - /commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/iterators/SingletonIterator.java

Author: ggregory
Date: Tue Nov 19 00:32:43 2013
New Revision: 1543244

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

Modified:
    commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/iterators/SingletonIterator.java

Modified: commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/iterators/SingletonIterator.java
URL: http://svn.apache.org/viewvc/commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/iterators/SingletonIterator.java?rev=1543244&r1=1543243&r2=1543244&view=diff
==============================================================================
--- commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/iterators/SingletonIterator.java (original)
+++ commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/iterators/SingletonIterator.java Tue Nov 19 00:32:43 2013
@@ -106,10 +106,9 @@ public class SingletonIterator<E>
         if (removeAllowed) {
             if (removed || beforeFirst) {
                 throw new IllegalStateException();
-            } else {
-                object = null;
-                removed = true;
             }
+            object = null;
+            removed = true;
         } else {
             throw new UnsupportedOperationException();
         }