JDK 10 RFR of JDK-8181396: Refactor shell test java/nio/file/Files/delete_on_close.sh to java
Amy Lu
amy.lu at oracle.com
Tue Jun 6 08:48:27 UTC 2017
On 6/6/17 3:46 PM, Alan Bateman wrote:
>
>
> On 06/06/2017 02:34, Amy Lu wrote:
>> On 6/5/17 10:52 PM, Alan Bateman wrote:
>>>> webrev: http://cr.openjdk.java.net/~amlu/8181396/webrev.00/
>>> I think it's a bit confusing to have both DeleteOnClose and
>>> DeleteOnTerminate, esp. when DeleteOnTerminate creates a file with
>>> DELETE_ON_CLOSE. Instead couldn't DeleteOnClose just launch itself
>>> to create the file?
>>
>> Please review the updated webrev:
>> http://cr.openjdk.java.net/~amlu/8181396/webrev.02/
>>
> This looks okay except for "/java/nio/file" in the @library value - I
> assume that is not needed.
This is for ../TestUtil.java, which is jdk/test/java/nio/file/TestUtil.java
Thank you for reviewing.
Thanks,
Amy
>
> -Alan
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/nio-dev/attachments/20170606/4696c832/attachment.html>
More information about the nio-dev
mailing list