On 25/04/2014 00:07, Mandy Chung wrote:
Thanks Sean.
I have updated the webrev:
http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8040059/webrev.01/
Erik - I'm including build-dev to review the build change for
java.policy file.
Just catching up on this thread.
The update to ZipFileSystem and the policy file look good to me.
I only skimmed over the test changes and agree with the the comments
that tests usually just want to configure the policy for the test and
not have to repeat the grants in the system policy.
A minor comment on the policy files is that there is mix of indentation
styles (4 and 8 in the same file in some cases). There is also
inconsistent indentation in Ext_AllPolicy.java (pre-dates your changes).
-Alan