You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@hc.apache.org by ol...@apache.org on 2010/03/24 20:46:33 UTC

svn commit: r927166 - /httpcomponents/httpasyncclient/trunk/src/main/java/org/apache/http/impl/nio/client/HttpExchangeImpl.java

Author: olegk
Date: Wed Mar 24 19:46:32 2010
New Revision: 927166

URL: http://svn.apache.org/viewvc?rev=927166&view=rev
Log:
Bug fix: future should be initialized before session lease
Contributed by Fabrice DELHOSTE <Fabrice.DELHOSTE at gemalto.com> 

Modified:
    httpcomponents/httpasyncclient/trunk/src/main/java/org/apache/http/impl/nio/client/HttpExchangeImpl.java

Modified: httpcomponents/httpasyncclient/trunk/src/main/java/org/apache/http/impl/nio/client/HttpExchangeImpl.java
URL: http://svn.apache.org/viewvc/httpcomponents/httpasyncclient/trunk/src/main/java/org/apache/http/impl/nio/client/HttpExchangeImpl.java?rev=927166&r1=927165&r2=927166&view=diff
==============================================================================
--- httpcomponents/httpasyncclient/trunk/src/main/java/org/apache/http/impl/nio/client/HttpExchangeImpl.java (original)
+++ httpcomponents/httpasyncclient/trunk/src/main/java/org/apache/http/impl/nio/client/HttpExchangeImpl.java Wed Mar 24 19:46:32 2010
@@ -55,8 +55,8 @@ class HttpExchangeImpl implements HttpEx
             final IOSessionManager<HttpRoute> sessmrg) {
         super();
         this.request = request;
-        this.sessionFuture = sessmrg.leaseSession(route, state, new InternalFutureCallback());
         this.responseFuture = new BasicFuture<HttpResponse>(null);
+        this.sessionFuture = sessmrg.leaseSession(route, state, new InternalFutureCallback());
     }
 
     public boolean isCompleted() {