On 01/03/2012 03:56, Konstantin Kolinko wrote: > 2012/3/1 <ma...@apache.org>: >> Author: markt >> Date: Wed Feb 29 21:24:59 2012 >> New Revision: 1295286
<snip/> >> Modified: tomcat/trunk/java/org/apache/catalina/websocket/WsFrame.java >> URL: >> http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/websocket/WsFrame.java?rev=1295286&r1=1295285&r2=1295286&view=diff >> ============================================================================== >> --- tomcat/trunk/java/org/apache/catalina/websocket/WsFrame.java (original) >> +++ tomcat/trunk/java/org/apache/catalina/websocket/WsFrame.java Wed Feb 29 >> 21:24:59 2012 >> @@ -215,8 +215,8 @@ public class WsFrame { >> } else if (read == 0) { >> return null; >> } else { >> - // TODO message >> - throw new IOException(); >> + throw new IOException( >> + sm.getString("frame.readFailed", >> Integer.valueOf(read))); > > Maybe java.io.EOFException ? > I see that read can be -1 here. Yes, read == -1 should really be handled as a separate case. Mark --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org