2014-10-17 20:26 GMT+04:00 Konstantin Kolinko <[email protected]>: > 2014-10-17 20:22 GMT+04:00 <[email protected]>: >> 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: [email protected] For additional commands, e-mail: [email protected]
