On 4/7/15 9:06 AM, Jaroslav Bachorik wrote:
Hi Mandy,
On 3.4.2015 23:35, Mandy Chung wrote:
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
Only the test parts updated -
http://cr.openjdk.java.net/~jbachorik/8074368/webrev.01/
Looks good.
thanks
Mandy
