<div dir="ltr"><div>Looks good; thx for reviewing other uses as well (and for finding the parallel compact issue of not splitting the components apart).<br><br></div>-- ramki<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">
On Mon, Oct 28, 2013 at 2:37 AM, Thomas Schatzl <span dir="ltr"><<a href="mailto:thomas.schatzl@oracle.com" target="_blank">thomas.schatzl@oracle.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi all,<br>
<br>
  can I have reviews for the following small change: it fixes the scope<br>
of some GCTraceTime timer, i.e. the braces that limited the scope to<br>
string table processing were missing.<br>
<br>
The change adds them.<br>
<br>
Bug:<br>
<a href="https://bugs.openjdk.java.net/browse/JDK-8027364" target="_blank">https://bugs.openjdk.java.net/browse/JDK-8027364</a><br>
<br>
Webrev:<br>
<a href="http://cr.openjdk.java.net/~tschatzl/8027364/webrev/" target="_blank">http://cr.openjdk.java.net/~tschatzl/8027364/webrev/</a><br>
<br>
Testing:<br>
jprt, compilation<br>
<br>
I had a look at other GCTraceTime uses in the code, and this seems to be<br>
the only case where the scope seems wrong.<br>
<br>
There is another place which could be considered buggy, that is in<br>
psParallelCompact.cpp, line 2114 where the GCTraceTime("class<br>
unloading"...) covers unloading, string table and symbol table<br>
unlinking. Other collectors seperate them out. I will probably create a<br>
separate CR for that if others agree that this is an issue.<br>
<br>
@Jeremy: We decided to try to get this fix in asap, possibly before we<br>
need to ask for exceptions to push, so I took over creating the patch.<br>
Jeremy, I hope you don't mind :) I will add the proper contribution.<br>
<span class="HOEnZb"><font color="#888888"><br>
Thomas<br>
<br>
<br>
</font></span></blockquote></div><br></div>