<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>