<html>
  <head>
    <meta http-equiv="Content-Type" content="text/html;
      charset=windows-1252">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    <br>
    <br>
    <div class="moz-cite-prefix">On 06/06/2018 20:26, Brian Burkhalter
      wrote:<br>
    </div>
    <blockquote type="cite"
      cite="mid:3DDB006C-71E1-4FF7-A014-9D239DC13150@oracle.com">
      <meta http-equiv="Content-Type" content="text/html;
        charset=windows-1252">
      On Jun 3, 2018, at 9:18 AM, Alan Bateman <<a
        href="mailto:Alan.Bateman@oracle.com" moz-do-not-send="true">Alan.Bateman@oracle.com</a>>
      wrote:<br>
      <div><br class="Apple-interchange-newline">
        <blockquote type="cite"><span style="font-family: Helvetica;
            font-size: 12px; font-style: normal; font-variant: normal;
            font-weight: normal; letter-spacing: normal; line-height:
            normal; orphans: auto; text-align: start; text-indent: 0px;
            text-transform: none; white-space: normal; widows: auto;
            word-spacing: 0px; -webkit-text-stroke-width: 0px; float:
            none; display: inline !important;">I've put a webrev with
            the updated patch here:</span><br style="font-family:
            Helvetica; font-size: 12px; font-style: normal;
            font-variant: normal; font-weight: normal; letter-spacing:
            normal; line-height: normal; orphans: auto; text-align:
            start; text-indent: 0px; text-transform: none; white-space:
            normal; widows: auto; word-spacing: 0px;
            -webkit-text-stroke-width: 0px;">
          <span style="font-family: Helvetica; font-size: 12px;
            font-style: normal; font-variant: normal; font-weight:
            normal; letter-spacing: normal; line-height: normal;
            orphans: auto; text-align: start; text-indent: 0px;
            text-transform: none; white-space: normal; widows: auto;
            word-spacing: 0px; -webkit-text-stroke-width: 0px; float:
            none; display: inline !important;">  <span
              class="Apple-converted-space"> </span></span><a
            href="http://cr.openjdk.java.net/%7Ealanb/6350055/webrev/index.html"
            style="font-family: Helvetica; font-size: 12px; font-style:
            normal; font-variant: normal; font-weight: normal;
            letter-spacing: normal; line-height: normal; orphans: auto;
            text-align: start; text-indent: 0px; text-transform: none;
            white-space: normal; widows: auto; word-spacing: 0px;
            -webkit-text-stroke-width: 0px;" moz-do-not-send="true">http://cr.openjdk.java.net/~alanb/6350055/webrev/index.html</a></blockquote>
      </div>
      <div><br>
      </div>
      <div>Looks fine modulo three picayune items:</div>
    </blockquote>
    Thanks. The CSR [1] has been approved so I'll try to get this pushed
    on Thursday.<br>
    <br>
    <blockquote type="cite"
      cite="mid:3DDB006C-71E1-4FF7-A014-9D239DC13150@oracle.com"><br>
      <div>1) In SelectionKey I recommend putting a comma after “e.g.”
        at line 245 and changing “rom” to “from” at line 246.</div>
    </blockquote>
    Okay.<br>
    <br>
    <br>
    <blockquote type="cite"
      cite="mid:3DDB006C-71E1-4FF7-A014-9D239DC13150@oracle.com">
      <div><br>
      </div>
      <div>2) In AtomicUpdates at line 179 I think the indentation is
        one space too deep.</div>
      <br>
    </blockquote>
    This is a try-with-resources with two resources. The second variable
    is just aligned with the first.<br>
    <br>
    -Alan<br>
    <br>
    [1] <a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8204472">https://bugs.openjdk.java.net/browse/JDK-8204472</a><br>
  </body>
</html>