On 10/07/2013 7:10 PM, Jaroslav Bachorik wrote:
Please, review this simple fix.
http://cr.openjdk.java.net/~jbachorik/8019826/webrev.00
Firstly, the patch removes a conditional early exit which checks for a
build 52 of an unspecified major JVM version - it is not needed any
more. Basically, the condition just made the test a noop till the latest
hotspot version.
The second fix is correctly setting the "mbean" attribute - it was not
properly initialized and because of this the test was going to fail with
NPE.
Looks fine to me.
David
-----
Thanks,
-JB-