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:36:30 UTC

svn commit: r1543246 - /commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/queue/CircularFifoQueue.java

Author: ggregory
Date: Tue Nov 19 00:36:29 2013
New Revision: 1543246

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

Modified:
    commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/queue/CircularFifoQueue.java

Modified: commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/queue/CircularFifoQueue.java
URL: http://svn.apache.org/viewvc/commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/queue/CircularFifoQueue.java?rev=1543246&r1=1543245&r2=1543246&view=diff
==============================================================================
--- commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/queue/CircularFifoQueue.java (original)
+++ commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/queue/CircularFifoQueue.java Tue Nov 19 00:36:29 2013
@@ -280,17 +280,15 @@ public class CircularFifoQueue<E> extend
     public E poll() {
         if (isEmpty()) {
             return null;
-        } else {
-            return remove();
         }
+        return remove();
     }
 
     public E element() {
         if (isEmpty()) {
             throw new NoSuchElementException("queue is empty");
-        } else {
-            return peek();
         }
+        return peek();
     }
 
     public E peek() {