Hi, Hamlin That looks pretty good. A couple comments:
* If I'm not mistaken, main/othervm/policy=... is sufficient to enable the default security manager, and specifying "-Djava.security.manager" is unnecessary.
* Please add 8180732 to the @bug tag You will also need approval from a JDK 10 Reviewer. Thanks, -Brent On 5/21/17 9:26 PM, Hamlin Li wrote:
Would you please review the below patch? bug: https://bugs.openjdk.java.net/browse/JDK-8180732 webrev: http://cr.openjdk.java.net/~mli/8180732/webrev.00/ Thank you -Hamlin
