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