svn commit: r763585 - /tomcat/trunk/java/org/apache/tomcat/util/net/JIoEndpoint.java

2009-04-09 Thread markt
Author: markt Date: Thu Apr 9 10:20:36 2009 New Revision: 763585 URL: http://svn.apache.org/viewvc?rev=763585view=rev Log: Java uses 0 rather than -1 for infinite socket timeout Modified: tomcat/trunk/java/org/apache/tomcat/util/net/JIoEndpoint.java Modified:

Re: svn commit: r763585 - /tomcat/trunk/java/org/apache/tomcat/util/net/JIoEndpoint.java

2009-04-09 Thread Remy Maucherat
On Thu, 2009-04-09 at 10:20 +, ma...@apache.org wrote: Author: markt Date: Thu Apr 9 10:20:36 2009 New Revision: 763585 URL: http://svn.apache.org/viewvc?rev=763585view=rev Log: Java uses 0 rather than -1 for infinite socket timeout But the value is never used if = 0, so what does it

Re: svn commit: r763585 - /tomcat/trunk/java/org/apache/tomcat/util/net/JIoEndpoint.java

2009-04-09 Thread Mark Thomas
Remy Maucherat wrote: On Thu, 2009-04-09 at 10:20 +, ma...@apache.org wrote: Author: markt Date: Thu Apr 9 10:20:36 2009 New Revision: 763585 URL: http://svn.apache.org/viewvc?rev=763585view=rev Log: Java uses 0 rather than -1 for infinite socket timeout But the value is never used

Re: svn commit: r763585 - /tomcat/trunk/java/org/apache/tomcat/util/net/JIoEndpoint.java

2009-04-09 Thread Remy Maucherat
On Thu, 2009-04-09 at 14:37 +0100, Mark Thomas wrote: This broke with http://svn.apache.org/viewvc?view=revrevision=703017 for org.apache.coyote.ajp.AjpProtocol but I suspect no-one ever tested it until today when I swapped AJP implementations for trunk. I'm open to fixing it a different