Looks good to me. (not a reviewer)
A small nit - in the test the copyright year should be only 2013
(instead of "2003, 2013")
-JB-
On 14.10.2013 14:44, Erik Gahlin wrote:
Hi,
Could I please have a review of this fix.
An empty string is returned if the thread name is null. The protocol
should be updated to support INVALID_STRING for the Thread Reference
Command <#Error_INVALID_STRING>, but that is a larger effort and
probably better to do when fixing other protocol changes. An empty
string is better than a crash.
Thanks
Erik
Webrev:
http://cr.openjdk.java.net/~egahlin/7105883/
Bug:
https://bugs.openjdk.java.net/browse/JDK-7105883
