Great, thanks! Changes look good to me.<br><br>reviewed.<br>-- ramki<br><br><div class="gmail_quote">On Fri, Jan 4, 2013 at 3:51 PM, Jon Masamitsu <span dir="ltr"><<a href="mailto:jon.masamitsu@oracle.com" target="_blank">jon.masamitsu@oracle.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im"><br>
<br>
On 1/4/2013 3:02 PM, Srinivas Ramakrishna wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi Jon, could you post examples of pre-unintended-change,<br>
post-unintended-change and post-this-fix? (couldn't find it in the visible<br>
part of the bug report)<br>
</blockquote></div>
Before unintended change (jdk7)<br>
<br>
[GC [ParNew: 69952K->8704K(78656K), 1.1970131 secs] 69952K->69906K(253440K), 1.1971601 secs] [Times: user=5.08 sys=0.78, real=1.20 secs]<br>
<br>
After unintended change - missing blank after "GC" (recent jdk8 promoted)<br>
<br>
[GC[ParNew: 69952K->8704K(78656K), 1.1672417 secs] 69952K->68461K(253440K), 1.1673949 secs] [Times: user=5.01 sys=0.75, real=1.17 secs]<br>
<br>
Fixed<br>
<br>
[GC [ParNew: 69952K->8704K(78656K), 1.1765617 secs] 69952K->68482K(253440K), 1.1767129 secs] [Times: user=5.06 sys=0.77, real=1.18 secs]<br>
<br>
Jon<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">
Happy New Year!<br>
-- ramki<br>
<br>
On Fri, Jan 4, 2013 at 2:03 PM, Jon Masamitsu<<a href="mailto:jon.masamitsu@oracle.com" target="_blank">jon.masamitsu@<u></u>oracle.com</a>>wrote:<br>
<br>
</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">
This is a clean up of some unintended white space changes in<br>
the GC logging output.<br>
<br>
</div><a href="http://cr.openjdk.java.net/%7E**jmasa/8005672/webrev.00/" target="_blank">http://cr.openjdk.java.net/~**<u></u>jmasa/8005672/webrev.00/</a><<a href="http://cr.openjdk.java.net/%7Ejmasa/8005672/webrev.00/" target="_blank">http:<u></u>//cr.openjdk.java.net/%<u></u>7Ejmasa/8005672/webrev.00/</a>><br>

<br>
Thanks.<br>
<br>
Jon<br>
<br>
<br>
</blockquote></blockquote>
</blockquote></div><br>