Hi Brian,

Looks good

Thanks, Roger


On 4/26/2017 4:57 PM, Brian Burkhalter wrote:
Hi Roger,

Here is an updated patch containing the suggested changes:

http://cr.openjdk.java.net/~bpb/8179247/webrev.01/ <http://cr.openjdk.java.net/%7Ebpb/8179247/webrev.01/>

Thanks,

Brian

On Apr 26, 2017, at 12:00 PM, Brian Burkhalter <brian.burkhal...@oracle.com <mailto:brian.burkhal...@oracle.com>> wrote:

On the original webrev:
In addition to checking if the directory exists would you want to list its contents?

Good idea.

And if checking the status of zpath, list its permission attributes?

I had thought of that as well but did not implement it. I can add that also.

Is there any chance that a previous delete of zpath has not completed, (since windows
has the delayed delete if the file is in use behavior).
I think can't think why that would be intermittent though.


Reply via email to