On 14/02/2018 01:23, yumin qi wrote:
Hi,

  I have update the webrev:
http://cr.openjdk.java.net/~minqi/8194154/webrev1/ <http://cr.openjdk.java.net/%7Eminqi/8194154/webrev1/>

  In this version, as suggested by Alan(thanks!), property of "user.dir" is cached and behave like it is 'read only'. The change made to *inux as well as windows. Since property of "user.dir" is cached, any changes via property setting for it has no effect.

This looks much better but you've removed a permission check from the Windows getUserPath implementation. That code the same permission check as the resolve method in the Unix implementation.

I think the test needs works. The simplest would be to call getCanonicalFile before changing the system property, then call it after and check that you an equal result. Also no need to hack the Properties object, you can use System.setProperty instead.

-Alan


Reply via email to