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() {