The code looks good to me (not an official reviewer).
However, I'm surprised this command doesn't
require the java.lang.management.ManagementPermission("monitor")
when invoked from the DiagnosticCommandMBean.
Has this topic been discussed during the CCC review
or with the security team?
Regards,
Fred
On 13/01/2014 14:17, Fredrik Arvidsson wrote:
Hi
Please help me review the following small enhancement:
Webrev: http://cr.openjdk.java.net/~farvidsson/8031304/webrev.00/
<http://cr.openjdk.java.net/%7Efarvidsson/8031304/webrev.00/>
Jira: https://bugs.openjdk.java.net/browse/JDK-8031304
/Fredrik
--
Frederic Parain - Oracle
Grenoble Engineering Center - France
Phone: +33 4 76 18 81 17
Email: [email protected]
