You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hc.apache.org by gg...@apache.org on 2015/08/27 00:02:22 UTC

svn commit: r1698013 - /httpcomponents/httpcore/trunk/httpcore/src/main/java/org/apache/http/concurrent/BasicFuture.java

Author: ggregory
Date: Wed Aug 26 22:02:21 2015
New Revision: 1698013

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

Modified:
    httpcomponents/httpcore/trunk/httpcore/src/main/java/org/apache/http/concurrent/BasicFuture.java

Modified: httpcomponents/httpcore/trunk/httpcore/src/main/java/org/apache/http/concurrent/BasicFuture.java
URL: http://svn.apache.org/viewvc/httpcomponents/httpcore/trunk/httpcore/src/main/java/org/apache/http/concurrent/BasicFuture.java?rev=1698013&r1=1698012&r2=1698013&view=diff
==============================================================================
--- httpcomponents/httpcore/trunk/httpcore/src/main/java/org/apache/http/concurrent/BasicFuture.java (original)
+++ httpcomponents/httpcore/trunk/httpcore/src/main/java/org/apache/http/concurrent/BasicFuture.java Wed Aug 26 22:02:21 2015
@@ -96,11 +96,10 @@ public class BasicFuture<T> implements F
                 wait(waitTime);
                 if (this.completed) {
                     return getResult();
-                } else {
-                    waitTime = msecs - (System.currentTimeMillis() - startTime);
-                    if (waitTime <= 0) {
-                        throw new TimeoutException();
-                    }
+                }
+                waitTime = msecs - (System.currentTimeMillis() - startTime);
+                if (waitTime <= 0) {
+                    throw new TimeoutException();
                 }
             }
         }