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

Reply via email to