Han Ming Ong wrote: > Thanks for at least hooking up the setter correctly. > > I know you folks are busy but if you can spend sometime thinking about > the default value (which you have currently set to false) when you have > a chance, we would truly appreciate it.
If you can give some good arguments why this value should be true and how it won't affect anything that works now - I'm sure we can change it. Costin > > Thanks again Costin. > > On Friday, October 4, 2002, at 04:33 PM, [EMAIL PROTECTED] wrote: > >> costin 2002/10/04 16:33:14 >> >> Modified: jk/java/org/apache/jk/common ChannelSocket.java >> Log: >> Fix 13270. tcpNoDelay wasn't implemented. >> >> Revision Changes Path >> 1.22 +4 -2 >> jakarta-tomcat-connectors/jk/java/org/apache/jk/common/ >> ChannelSocket.java >> >> Index: ChannelSocket.java >> =================================================================== >> RCS file: >> /home/cvs/jakarta-tomcat-connectors/jk/java/org/apache/jk/common/ >> ChannelSocket.java,v >> retrieving revision 1.21 >> retrieving revision 1.22 >> diff -u -r1.21 -r1.22 >> --- ChannelSocket.java 9 Aug 2002 20:52:37 -0000 1.21 >> +++ ChannelSocket.java 4 Oct 2002 23:33:14 -0000 1.22 >> @@ -105,7 +105,7 @@ >> int port=startPort; >> InetAddress inet; >> int serverTimeout; >> - boolean tcpNoDelay; >> + boolean tcpNoDelay=false; >> int linger=100; >> int socketTimeout; >> >> @@ -259,7 +259,9 @@ >> s.setSoLinger( true, linger); >> if( socketTimeout > 0 ) >> s.setSoTimeout( socketTimeout ); >> - >> + if( tcpNoDelay ) >> + s.setTcpNoDelay( true ); >> + >> requestCount++; >> >> InputStream is=new BufferedInputStream(s.getInputStream()); >> >> >> >> >> -- >> To unsubscribe, e-mail: >> <mailto:[EMAIL PROTECTED]> >> For additional commands, e-mail: >> <mailto:[EMAIL PROTECTED]> >> -- Costin -- To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]> For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
