Hi Hamlin,

ClassPathCodebase:
 - line 51: The comment says seconds but it is really milliseconds.

security.policy:
 - line 21: "this its" -> "its"

 - line 26:  Is the relative path in the File permission correct;
   Is there a ${test.xxx} path to refer to that file.


registry.security.policy: Add a comment saying how this policy is used (to distinguish it from the other security policy).

Otherwise looks fine,

Roger


On 1/11/2017 11:33 PM, Hamlin Li wrote:
Would you please review the below patch?

bug: https://bugs.openjdk.java.net/browse/JDK-8030950
webrev: http://cr.openjdk.java.net/~mli/8030950/webrev.00/

Thank you
-Hamlin

Reply via email to