You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by ma...@apache.org on 2015/01/09 13:18:52 UTC
svn commit: r1650510 -
/tomcat/trunk/java/org/apache/tomcat/util/net/AprEndpoint.java
Author: markt
Date: Fri Jan 9 12:18:51 2015
New Revision: 1650510
URL: http://svn.apache.org/r1650510
Log:
Better TODO
Modified:
tomcat/trunk/java/org/apache/tomcat/util/net/AprEndpoint.java
Modified: tomcat/trunk/java/org/apache/tomcat/util/net/AprEndpoint.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/net/AprEndpoint.java?rev=1650510&r1=1650509&r2=1650510&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/tomcat/util/net/AprEndpoint.java (original)
+++ tomcat/trunk/java/org/apache/tomcat/util/net/AprEndpoint.java Fri Jan 9 12:18:51 2015
@@ -2377,6 +2377,8 @@ public class AprEndpoint extends Abstrac
public AprSocketWrapper(Long socket, AprEndpoint endpoint) {
super(socket, endpoint);
+ // TODO Make the socketWriteBuffer size configurable and align the
+ // SSL and app buffer size settings with NIO & NIO2.
if (endpoint.isSSLEnabled()) {
sslOutputBuffer = ByteBuffer.allocateDirect(SSL_OUTPUT_BUFFER_SIZE);
sslOutputBuffer.position(SSL_OUTPUT_BUFFER_SIZE);
@@ -2384,7 +2386,6 @@ public class AprEndpoint extends Abstrac
sslOutputBuffer = null;
}
- // TODO: This needs to be expandable to the header buffer size
socketWriteBuffer = ByteBuffer.allocateDirect(6 * 1500);
}
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org