<html>
  <head>
    <meta content="text/html; charset=utf-8" http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    <div class="moz-cite-prefix">On 6/6/17 3:46 PM, Alan Bateman wrote:<br>
    </div>
    <blockquote
      cite="mid:70f6f3f7-6fed-4edd-e1ff-786eebcd12a5@oracle.com"
      type="cite">
      <meta http-equiv="Content-Type" content="text/html; charset=utf-8">
      <br>
      <br>
      <div class="moz-cite-prefix">On 06/06/2017 02:34, Amy Lu wrote:<br>
      </div>
      <blockquote type="cite"
        cite="mid:c11eb839-eeda-7a3b-2ffe-7597a32a37e4@oracle.com">
        <meta content="text/html; charset=utf-8"
          http-equiv="Content-Type">
        <div class="moz-cite-prefix">On 6/5/17 10:52 PM, Alan Bateman
          wrote:<br>
        </div>
        <blockquote
          cite="mid:77f166d0-5a37-e9ef-b9df-be8b5c20a217@oracle.com"
          type="cite">
          <blockquote type="cite" style="font-size: large; color:
            #000000;">webrev: <a moz-do-not-send="true"
              class="moz-txt-link-freetext"
              href="http://cr.openjdk.java.net/%7Eamlu/8181396/webrev.00/">http://cr.openjdk.java.net/~amlu/8181396/webrev.00/</a>
            <br>
          </blockquote>
          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? </blockquote>
        <br>
        <tt>Please review the updated webrev:</tt><tt><br>
        </tt><tt><a class="moz-txt-link-freetext"
            href="http://cr.openjdk.java.net/%7Eamlu/8181396/webrev.02/"
            moz-do-not-send="true">http://cr.openjdk.java.net/~amlu/8181396/webrev.02/</a></tt><tt><br>
        </tt><br>
      </blockquote>
      This looks okay except for "/java/nio/file" in the @library value
      - I assume that is not needed.<br>
    </blockquote>
    <tt>This is for ../TestUtil.java, which is
      jdk/test/java/nio/file/TestUtil.java</tt><tt><br>
      <br>
      Thank you for reviewing.<br>
    </tt><tt><br>
    </tt><tt>Thanks,</tt><tt><br>
    </tt><tt>Amy</tt><br>
    <blockquote
      cite="mid:70f6f3f7-6fed-4edd-e1ff-786eebcd12a5@oracle.com"
      type="cite"> <br>
      -Alan <br>
    </blockquote>
    <br>
  </body>
</html>