Good!

Thanks
Max

On 9/13/2016 21:53, Sibabrata Sahoo wrote:
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; [email protected]
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



Reply via email to