RFR: 6588467: Add isDaemon() and getPriority() to ThreadInfo

Mandy Chung mandy.chung at oracle.com
Mon Feb 16 23:44:37 UTC 2015


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




More information about the serviceability-dev mailing list