On 4/3/15 7:31 AM, Jaroslav Bachorik wrote:
Please, review the following change

Issue : https://bugs.openjdk.java.net/browse/JDK-8074368
Webrev: http://cr.openjdk.java.net/~jbachorik/8074368/webrev.00

Looks okay to me.

Minor comment on ThreadInfoArray.java test:
You can have one single @bug line with multiple bug numbers
line 57, 90: can you rename "direct" to mbean?
line 59, 92: formatting - indent to the right

Mandy


Reply via email to