<html>
  <head>
    <meta http-equiv="Content-Type" content="text/html;
      charset=windows-1252">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    On 15/06/2018 21:31, Brian Burkhalter wrote:<br>
    <blockquote type="cite"
      cite="mid:4451E039-7239-4CAC-9229-84D346B079F4@oracle.com"><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;">
      <br>
      <div>Minor comments on the SelectWithConsumer test:</div>
      <div><br>
      </div>
      <div>56:<span class="Apple-tab-span" style="white-space:pre">     </span>checks
        -> check</div>
      <div>538:<span class="Apple-tab-span" style="white-space:pre">    </span>removes
        -> remove</div>
      <div><br>
      </div>
      <div>In the assertTrue() calls which involve a duration check,
        might it be useful to print the value of ‘duration’ if the
        assertion fails?</div>
      <div><br>
      </div>
    </blockquote>
    Good idea. I'll change these asserts from `assertTrue(e)` to
    `assertTrue(e, "select took " + duration + " ms")` so the jtr file
    has the duration in the event of failure. I've run this test on all
    platforms several hundred times (with high concurrency) so I'm not
    expecting any issues but it is always tricky to test timed selection
    operations.<br>
    <br>
    <blockquote type="cite"
      cite="mid:4451E039-7239-4CAC-9229-84D346B079F4@oracle.com">
      <div>I think that this can go forward without further review if
        the foregoing items are changed.</div>
      <br>
    </blockquote>
    Thanks, I'll get this pushed so that it's in jdk-11-ea+19.<br>
    <br>
    -Alan<br>
  </body>
</html>