<html>
  <head>
    <meta content="text/html; charset=ISO-8859-1"
      http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    Hi Marcus!<br>
    <br>
    While it's not critical and not really connected to the issue you're
    solving, wouldn't it be better to replace<br>
    <pre style="color: rgb(0, 0, 0); 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; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; background-color: rgb(238, 238, 238);">344     p = utf8_write(p, base[index]);</pre>
    with<br>
    <pre style="color: rgb(0, 0, 0); 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; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; background-color: rgb(238, 238, 238);">344     p = utf8_write(p, <b>c</b>);
</pre>
    in src/share/vm/utilities/utf8.cpp?<br>
    <br>
    Sincerely yours,<br>
    Ivan<br>
    <br>
    <br>
    <div class="moz-cite-prefix">On 03.04.2014 17:17, Marcus Larsson
      wrote:<br>
    </div>
    <blockquote cite="mid:533D5F54.4060202@oracle.com" type="cite">Hello,
      <br>
      <br>
      I would like reviews for the following bugfix removing a redundant
      memcpy.
      <br>
      Built on all platforms and tested with JCK.
      <br>
      <br>
      Webrev:
      <br>
      <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~dsimms/6664815/">http://cr.openjdk.java.net/~dsimms/6664815/</a>
      <br>
      <br>
      Bug:
      <br>
      <a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-6664815">https://bugs.openjdk.java.net/browse/JDK-6664815</a>
      <br>
      <br>
      Regards,
      <br>
      Marcus
      <br>
      <br>
      <br>
    </blockquote>
    <br>
  </body>
</html>