Mandy Chung wrote:
Alan, Sean,
Can you review the fix for:
6891707: Eliminate the java.io.FilePermission dependency on PolicyFile
Webrev:
http://cr.openjdk.java.net/~mchung/6891707/webrev.00/
FilePermission and PolicyFile are the two callers to the
sun.security.provider.PolicyFile.canonPath method. Adding a new class
for the canonPath method seems unnecessary. So the fix just
duplicates the code in FilePermission class.
Thanks
Mandy
This looks okay to me. Minor comment is this is just a wrapper about
getCanonicalPath to avoid calling it with paths that are intended to
match all entries in the directory. It might be good to include this in
a comment in both places (and this would avoid needing to say that it
duplicated in FilePermission).
-Alan