Author: norman
Date: Tue Sep 27 06:19:41 2011
New Revision: 1176225

URL: http://svn.apache.org/viewvc?rev=1176225&view=rev
Log:
Make sure the ResponseListener will be notified with a FutureResponse

Modified:
    
james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/FutureResponse.java
    
james/protocols/trunk/impl/src/main/java/org/apache/james/protocols/impl/NettyProtocolTransport.java
    
james/protocols/trunk/smtp/src/main/java/org/apache/james/protocols/smtp/core/DataLineMessageHookHandler.java

Modified: 
james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/FutureResponse.java
URL: 
http://svn.apache.org/viewvc/james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/FutureResponse.java?rev=1176225&r1=1176224&r2=1176225&view=diff
==============================================================================
--- 
james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/FutureResponse.java
 (original)
+++ 
james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/FutureResponse.java
 Tue Sep 27 06:19:41 2011
@@ -57,10 +57,10 @@ public interface FutureResponse extends 
     public interface ResponseListener {
 
         /**
-         * The {@link Response} is ready for processing
+         * The {@link FutureResponse} is ready for processing
          * 
          * @param response
          */
-        public void onResponse(Response response);
+        public void onResponse(FutureResponse response);
     }
 }

Modified: 
james/protocols/trunk/impl/src/main/java/org/apache/james/protocols/impl/NettyProtocolTransport.java
URL: 
http://svn.apache.org/viewvc/james/protocols/trunk/impl/src/main/java/org/apache/james/protocols/impl/NettyProtocolTransport.java?rev=1176225&r1=1176224&r2=1176225&view=diff
==============================================================================
--- 
james/protocols/trunk/impl/src/main/java/org/apache/james/protocols/impl/NettyProtocolTransport.java
 (original)
+++ 
james/protocols/trunk/impl/src/main/java/org/apache/james/protocols/impl/NettyProtocolTransport.java
 Tue Sep 27 06:19:41 2011
@@ -92,7 +92,7 @@ public class NettyProtocolTransport impl
         if (response instanceof FutureResponse) {
             ((FutureResponse) response).addListener(new ResponseListener() {
                 
-                public void onResponse(Response response) {
+                public void onResponse(FutureResponse response) {
                     writeResponseToChannel(response, session);                 
   
                 }
             });

Modified: 
james/protocols/trunk/smtp/src/main/java/org/apache/james/protocols/smtp/core/DataLineMessageHookHandler.java
URL: 
http://svn.apache.org/viewvc/james/protocols/trunk/smtp/src/main/java/org/apache/james/protocols/smtp/core/DataLineMessageHookHandler.java?rev=1176225&r1=1176224&r2=1176225&view=diff
==============================================================================
--- 
james/protocols/trunk/smtp/src/main/java/org/apache/james/protocols/smtp/core/DataLineMessageHookHandler.java
 (original)
+++ 
james/protocols/trunk/smtp/src/main/java/org/apache/james/protocols/smtp/core/DataLineMessageHookHandler.java
 Tue Sep 27 06:19:41 2011
@@ -28,7 +28,6 @@ import java.util.List;
 
 import org.apache.james.protocols.api.FutureResponse;
 import org.apache.james.protocols.api.FutureResponse.ResponseListener;
-import org.apache.james.protocols.api.Response;
 import org.apache.james.protocols.api.handler.ExtensibleHandler;
 import org.apache.james.protocols.api.handler.LineHandler;
 import org.apache.james.protocols.api.handler.WiringException;
@@ -73,7 +72,7 @@ public class DataLineMessageHookHandler 
                 if (response instanceof FutureResponse) {
                     ((FutureResponse) response).addListener(new 
ResponseListener() {
                         
-                        public void onResponse(Response response) {
+                        public void onResponse(FutureResponse response) {
                             session.resetState();
                         }
                     });



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to