Statement unnecessarily nested within else clause.

git-svn-id: https://svn.apache.org/repos/asf/httpcomponents/httpclient/trunk@1726954 13f79535-47bb-0310-9956-ffa450edef68
cash · Jan 27, 2016 · 69b509a · 69b509a
1 parent 5868cee
commit 69b509a
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/httpclient5/src/main/java/org/apache/hc/client5/http/impl/sync/MinimalClientExec.java b/httpclient5/src/main/java/org/apache/hc/client5/http/impl/sync/MinimalClientExec.java
@@ -145,9 +145,8 @@ public CloseableHttpResponse execute(
             if (execAware.isAborted()) {
                 connRequest.cancel();
                 throw new RequestAbortedException("Request aborted");
-            } else {
-                execAware.setCancellable(connRequest);
             }
+            execAware.setCancellable(connRequest);
         }
 
         final RequestConfig config = context.getRequestConfig();
@@ -228,9 +227,8 @@ public CloseableHttpResponse execute(
                 // connection not needed and (assumed to be) in re-usable state
                 releaseTrigger.releaseConnection();
                 return new HttpResponseProxy(response, null);
-            } else {
-                return new HttpResponseProxy(response, releaseTrigger);
             }
+            return new HttpResponseProxy(response, releaseTrigger);
         } catch (final ConnectionShutdownException ex) {
             final InterruptedIOException ioex = new InterruptedIOException(
                     "Connection has been shut down");