Re: svn commit: r1360905 - /tomcat/trunk/java/org/apache/tomcat/util/net/NioSelectorPool.java

2012-07-12 Thread Mark Thomas
On 12/07/2012 21:11, fha...@apache.org wrote: Author: fhanik Date: Thu Jul 12 20:11:31 2012 New Revision: 1360905 URL: http://svn.apache.org/viewvc?rev=1360905view=rev Log: Correct handling of timeout - negative or zero means no timeout but an instant Nope. The expected and documented

RE: svn commit: r1360905 - /tomcat/trunk/java/org/apache/tomcat/util/net/NioSelectorPool.java

2012-07-12 Thread Filip Hanik (mailing lists)
tempted to get rid of the -1 notion all together, no sane person should ever use no timeout :) -Original Message- From: Mark Thomas [mailto:ma...@apache.org] Sent: Thursday, July 12, 2012 2:20 PM To: Tomcat Developers List Subject: Re: svn commit: r1360905 - /tomcat/trunk/java/org

Re: svn commit: r1360905 - /tomcat/trunk/java/org/apache/tomcat/util/net/NioSelectorPool.java

2012-07-12 Thread Mark Thomas
On 12/07/2012 21:42, Filip Hanik (mailing lists) wrote: You are correct, I was chasing down the following: Testsuite: org.apache.catalina.websocket.TestWebSocket Tests run: 6, Failures: 1, Errors: 0, Time elapsed: 2.048 sec INFO: Starting ProtocolHandler