<html>
  <head>
    <meta http-equiv="Content-Type" content="text/html; charset=utf-8">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    <br>
    <br>
    <div class="moz-cite-prefix">On 26/06/2018 18:41, Joe Wang wrote:<br>
    </div>
    <blockquote type="cite" cite="mid:5B327ACF.3080804@oracle.com">
      <br>
      :<br>
      <br>
      Yes, combined with Sherman's suggestion eliminated the need for
      the new parameter. Here's the updated webrev:
      <br>
      <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~joehw/jdk11/8205058/webrev03/">http://cr.openjdk.java.net/~joehw/jdk11/8205058/webrev03/</a>
      <br>
    </blockquote>
    This looks good.<br>
    <br>
    Just a minor nit but newStringNoRepl and getBytesNoReply don't need
    to check if the cause is null.<br>
    <br>
    -Alan<br>
    <span class="changed" style="color: blue;"></span>
  </body>
</html>