<html>
<head>
<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
Hi Thomas,<br>
<br>
Thanks you for reviewing diff :)<br>
<br>
Chihiro<br>
<br>
<div class="moz-cite-prefix">On 2017/04/11 23:02, Thomas Schatzl
wrote:<br>
</div>
<blockquote cite="mid:1491919338.2754.58.camel@oracle.com"
type="cite">
<pre wrap="">Hi Chihiro,
On Tue, 2017-04-11 at 03:04 +0900, chihiro ito wrote:
</pre>
<blockquote type="cite">
<pre wrap="">Hi all,
Could I have a reviews for this fix, please? This patch add spaces
following each ":" like a JDK 8 log in log message.
Log message that ran JDK 9 with -Xlog:gc+ergo+heap=debug option is
following.
[0.019s][debug][gc,ergo,heap] Expand the heap. requested expansion
amount: 10485760B expansion amount: 10485760B
Log message that ran JDK 8 with -XX:+PrintAdaptiveSizePolicy option
is following.
0.124: [G1Ergonomics (Heap Sizing) expand the heap, requested
expansion amount: 10485760 bytes, attempted expansion amount:
10485760 bytes]
Bug:
Missing spaces in log message
<a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8166191">https://bugs.openjdk.java.net/browse/JDK-8166191</a>
</pre>
</blockquote>
<pre wrap="">
I also noticed that just a few days ago. Thanks for fixing this :)
I created a webrev from your diff at
<a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~tschatzl/8166191/webrev/">http://cr.openjdk.java.net/~tschatzl/8166191/webrev/</a>
Looks good. I can sponsor.
Passes jprt.
Thanks,
Thomas
</pre>
</blockquote>
<br>
<div class="moz-signature">-- <br>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
<title></title>
<p>
<font size="2" color="#666666" face="Verdana, Arial, Helvetica,
sans-serif"> Chihiro Ito | Principal Consultant |
+81.90.6148.8815<br>
<a href="http://www.oracle.com" target="_blank"><font
color="#FF0000">Oracle</font></a> Consultant<br>
ORACLE Japan | Akasaka Center Bldg. | Motoakasaka 1-3-13 |
1070051 Minato-ku, Tokyo, JAPAN</font>
</p>
<a href="http://www.oracle.com/commitment" target="_blank">
<font size="1" color="#4B7D42" face="Verdana, Arial, Helvetica,
sans-serif">Oracle is committed to developing practices and
products that help protect the environment</font>
</a>
</div>
</body>
</html>