Hi Jaroslav,

This movement of functionality seems okay to me. And good to see the JDK 6 code removed :)

Thanks,
David

On 15/12/2015 2:55 AM, Jaroslav Bachorik wrote:
Please, review the following change

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

The problem is that the class UsageTrackerClient is accessing
RuntimeMXBean.getInputArguments() method via reflection to avoid static
dependency. However, with functional module boundaries this fails.

Since the functionality provided by RuntimeMXBean.getInputArguments() is
not, in fact, specific to the management subsystem, it makes sense to
move it out of jmm.h to jvm.h

Full suite of SVC tests was run against this change successfully on all
platforms.


Thanks,

-JB-

Reply via email to