Hi Max, Here is the updated webrev: http://cr.openjdk.java.net/~ssahoo/8156054/webrev.01/
Thanks, Siba -----Original Message----- From: Weijun Wang Sent: Tuesday, September 13, 2016 4:25 PM To: Sibabrata Sahoo; security-dev@openjdk.java.net Subject: Re: [9] RFR: 8156054: Test Task: Develop new tests for JEP C155: Remove FilePermission Pathname Canonicalization 44 final String current = "."; 45 final File realFile = new File(current, "exist.file"); I suggest we just use new File("exist.file"). No other comment. Thanks Max On 9/13/2016 18:14, Sibabrata Sahoo wrote: > Hi, > > > > Please review the patch for, > > > > JBS: https://bugs.openjdk.java.net/browse/JDK-8156054 > > Webrev: http://cr.openjdk.java.net/~ssahoo/8156054/webrev.00/ > > > > Description: > > It checks equals(), implies() and hashCode () of FilePermission when > "jdk.io.permissionsUseCanonicalPath" set and un-set. Along with, it > also verify compatibility with previous JDK version. > > > > > > Thanks, > > Siba > > >