Hi Jeremy,

On 2/9/2015 4:51 PM, Jeremy Manson wrote:
http://cr.openjdk.java.net/~jmanson/6588467/webrev.01/ <http://cr.openjdk.java.net/%7Ejmanson/6588467/webrev.01/>


The change looks okay to me.

Nit: It would be good for the new methods to replace <tt>...</tt> with {@code ...}. line 600, 603 {@code ThreadInfo}. It would be good to add {@linkplain Thread#isDaemon daemon thread} or @see Thread#isDaemon . Similarly Thread#getPriority.

thanks
Mandy


Reply via email to