svn commit: r1176240 - in /james/protocols/trunk/impl/src/main/java/org/apache/james/protocols/impl: BasicChannelUpstreamHandler.java LineHandlerUpstreamHandler.java

[norman]

Author: norman
Date: Tue Sep 27 07:11:02 2011
New Revision: 1176240

URL: http://svn.apache.org/viewvc?rev=1176240&view=rev
Log:
Add TODO

Modified:
    
james/protocols/trunk/impl/src/main/java/org/apache/james/protocols/impl/BasicChannelUpstreamHandler.java
    
james/protocols/trunk/impl/src/main/java/org/apache/james/protocols/impl/LineHandlerUpstreamHandler.java

Modified: 
james/protocols/trunk/impl/src/main/java/org/apache/james/protocols/impl/BasicChannelUpstreamHandler.java
URL: 
http://svn.apache.org/viewvc/james/protocols/trunk/impl/src/main/java/org/apache/james/protocols/impl/BasicChannelUpstreamHandler.java?rev=1176240&r1=1176239&r2=1176240&view=diff
==============================================================================
--- 
james/protocols/trunk/impl/src/main/java/org/apache/james/protocols/impl/BasicChannelUpstreamHandler.java
 (original)
+++ 
james/protocols/trunk/impl/src/main/java/org/apache/james/protocols/impl/BasicChannelUpstreamHandler.java
 Tue Sep 27 07:11:02 2011
@@ -102,6 +102,7 @@ public class BasicChannelUpstreamHandler
                     resultHandlers.get(a).onResponse(session, response, 
executionTime, cHandler);
                 }
                 if (response != null) {
+                    // TODO: This kind of sucks but I was able to come up with 
something more elegant here
                     
((AbstractSession)session).getProtocolTransport().writeResponse(response, 
session);
                 }
                
@@ -158,6 +159,7 @@ public class BasicChannelUpstreamHandler
                 resultHandlers.get(i).onResponse(pSession, response, 
executionTime, lHandler);
             }
             if (response != null) {
+                // TODO: This kind of sucks but I was able to come up with 
something more elegant here
                 
((AbstractSession)pSession).getProtocolTransport().writeResponse(response, 
pSession);
             }
 

Modified: 
james/protocols/trunk/impl/src/main/java/org/apache/james/protocols/impl/LineHandlerUpstreamHandler.java
URL: 
http://svn.apache.org/viewvc/james/protocols/trunk/impl/src/main/java/org/apache/james/protocols/impl/LineHandlerUpstreamHandler.java?rev=1176240&r1=1176239&r2=1176240&view=diff
==============================================================================
--- 
james/protocols/trunk/impl/src/main/java/org/apache/james/protocols/impl/LineHandlerUpstreamHandler.java
 (original)
+++ 
james/protocols/trunk/impl/src/main/java/org/apache/james/protocols/impl/LineHandlerUpstreamHandler.java
 Tue Sep 27 07:11:02 2011
@@ -57,6 +57,7 @@ public class LineHandlerUpstreamHandler<
 
         Response response = handler.onLine(session, line); 
         if (response != null) {
+            // TODO: This kind of sucks but I was able to come up with 
something more elegant here
             
((AbstractSession)session).getProtocolTransport().writeResponse(response, 
session);
         }
     }



---------------------------------------------------------------------
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org