2014-10-17 20:26 GMT+04:00 Konstantin Kolinko <knst.koli...@gmail.com>: > 2014-10-17 20:22 GMT+04:00 <r...@apache.org>: >> Author: remm >> Date: Fri Oct 17 16:22:43 2014 >> New Revision: 1632625 >> >> URL: http://svn.apache.org/r1632625 >> Log: >> Add null checks for arguments in remote endpoint. >> > > Wouldn't it better to throw a NullPointerException with those messages?
Correcting myself: The JavaEE javadoc [1] says that null data results in IllegalArgumentException. So this change is a correct one. [1] http://docs.oracle.com/javaee/7/api/javax/websocket/RemoteEndpoint.Basic.html Best regards, Konstantin Kolinko --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org