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 2007/03/04 14:01:47 UTC

svn commit: r514394 - /jakarta/httpcomponents/httpcore/trunk/module-nio/src/main/java/org/apache/http/impl/nio/DefaultNHttpClientConnection.java

Author: olegk
Date: Sun Mar  4 05:01:46 2007
New Revision: 514394

URL: http://svn.apache.org/viewvc?view=rev&rev=514394
Log:
HTTPCORE-51: Fixed problem with NHttpClientHandler#requestReady() firing unnecessarily

Modified:
    jakarta/httpcomponents/httpcore/trunk/module-nio/src/main/java/org/apache/http/impl/nio/DefaultNHttpClientConnection.java

Modified: jakarta/httpcomponents/httpcore/trunk/module-nio/src/main/java/org/apache/http/impl/nio/DefaultNHttpClientConnection.java
URL: http://svn.apache.org/viewvc/jakarta/httpcomponents/httpcore/trunk/module-nio/src/main/java/org/apache/http/impl/nio/DefaultNHttpClientConnection.java?view=diff&rev=514394&r1=514393&r2=514394
==============================================================================
--- jakarta/httpcomponents/httpcore/trunk/module-nio/src/main/java/org/apache/http/impl/nio/DefaultNHttpClientConnection.java (original)
+++ jakarta/httpcomponents/httpcore/trunk/module-nio/src/main/java/org/apache/http/impl/nio/DefaultNHttpClientConnection.java Sun Mar  4 05:01:46 2007
@@ -125,6 +125,11 @@
     }
 
     public void produceOutput(final NHttpClientHandler handler) {
+
+        if (this.request == null && !this.closed) {
+            handler.requestReady(this);
+        }
+        
         try {
             if (this.outbuf.hasData()) {
                 this.outbuf.flush(this.session.channel());
@@ -151,10 +156,6 @@
         } finally {
             // Finally set buffered output flag
             this.hasBufferedOutput = this.outbuf.hasData();
-        }
-
-        if (this.request == null && !this.closed) {
-            handler.requestReady(this);
         }
     }