<html>
  <head>
    <meta content="text/html; charset=utf-8" http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    <br>
    <br>
    <div class="moz-cite-prefix">On 15/01/2016 16:54, Tony Printezis
      wrote:<br>
    </div>
    <blockquote cite="mid:etPan.5699242a.5225d31c.58e@tw-mbp-tprintezis"
      type="cite">
      <style>body{font-family:Helvetica,Arial;font-size:13px}</style>
      <div id="bloop_customfont"
        style="font-family:Helvetica,Arial;font-size:13px; color:
        rgba(0,0,0,1.0); margin: 0px; line-height: auto;">(I’m starting
        a new thread so that it’s obvious this is a code review request)
        Second version of this change, addressing Alan’s comments (see
        previous e-mail exchange):</div>
      <div id="bloop_customfont"
        style="font-family:Helvetica,Arial;font-size:13px; color:
        rgba(0,0,0,1.0); margin: 0px; line-height: auto;"><br>
      </div>
      <div id="bloop_customfont"
        style="font-family:Helvetica,Arial;font-size:13px; color:
        rgba(0,0,0,1.0); margin: 0px; line-height: auto;"><a
          moz-do-not-send="true"
          href="http://cr.openjdk.java.net/%7Etonyp/nio-max-buffer-size/webrev.2/"><a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~tonyp/nio-max-buffer-size/webrev.2/">http://cr.openjdk.java.net/~tonyp/nio-max-buffer-size/webrev.2/</a></a></div>
      <div id="bloop_customfont"
        style="font-family:Helvetica,Arial;font-size:13px; color:
        rgba(0,0,0,1.0); margin: 0px; line-height: auto;"><br>
      </div>
      <br>
    </blockquote>
    I think this mostly looks okay. I assume the property should be
    jdk.nio.maxCachedBufferSize so that the casing is consistent with
    other properties. As this will be documented/supported property then
    I'll need to get a CCC submitted.<br>
    <br>
    A few comments on the test:<br>
    <br>
    1. The copyright headers needs to be the pure GPL header, we don't
    use the Classpath exception on tests. Also the date is duplicated.<br>
    <br>
    2. Using BufferOverflowException when filling the buffer with random
    bytes is a bit icky, it could use while buffer.hasRemaining()
    instead. Alternatively random.nextBytes(buffer.array()) should do it
    in one-line.<br>
    <br>
    3. If you want then you could create the FileChannel directly, the
    RandomAccessFile + getChannel is not needed.<br>
    <br>
    I think that's it.<br>
    <br>
    -Alan.<br>
    <br>
    <br>
    <br>
  </body>
</html>