jdk10 RFR of JDK-8180732: add test to check temp file permission
Brent Christian
brent.christian at oracle.com
Tue May 23 16:53:26 UTC 2017
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
More information about the core-libs-dev
mailing list