On May 27, 2014, at 6:16 PM, Vladimir Ivanov <[email protected]> 
wrote:

> Good catch!
> One more iteration:
> http://cr.openjdk.java.net/~vlivanov/8035186/webrev.02/
> 

Looks good.

Some minor points, no need for another review round either way:

- don't need the nested try blocks in isWriteableDirectory, just tack the 
finally block on the end of the last catch block.

- could add a println when there is no POSIX environment to indicate the test 
is not run and remove it when/if jtreg supports skipping.

Paul.


Reply via email to