Looks good. Didn't realize that mark_and_follow() had been dead for so long! :-)<br><br>reviewed!<br>-- ramki (ysr)<br><br><div class="gmail_quote">On Mon, Nov 28, 2011 at 10:24 AM, Jon Masamitsu <span dir="ltr"><<a href="mailto:jon.masamitsu@oracle.com">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;">Bengt,<br>
<br>
Thanks for the review.  I've made your suggested changes and<br>
also removed some additional dead code (unused mark_and_follow()<br>
methods) suggested by another reviewer.  New webrev is at<br>
<br>
<a href="http://cr.openjdk.java.net/%7Ejmasa/7112997/webrev.01" target="_blank">http://cr.openjdk.java.net/~<u></u>jmasa/7112997/webrev.01</a><span class="HOEnZb"><font color="#888888"><br>
<br>
Jon</font></span><div class="HOEnZb"><div class="h5"><br>
<br>
On 11/27/11 23:38, Bengt Rutisson wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Jon,<br>
<br>
I think you removed the only calls to ParCompactionManager::should_<u></u>verify_only() and ParCompactionManager::should_<u></u>reset_only(). Can you remove those methods as well? That will propagate to removing the enum values VerifyUpdate and ResetObjects in ParCompactionManager::Action.<br>

<br>
Other than that it looks good.<br>
<br>
Thanks for doing cleanups like this one!<br>
Bengt<br>
<br>
On 2011-11-23 20:24, Jon Masamitsu wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
7112997: Remove obsolete code ResetObjectsClosure and VerifyUpdateClosure<br>
Summary: Remove obsolete code.<br>
<br>
This was debugging code that used  the MarkSweep code to check<br>
the UseParallelOldGC  compaction.   It has been broken for sometime<br>
and there are no prospects for fixing it.<br>
<br>
<a href="http://cr.openjdk.java.net/%7Ejmasa/7112997/webrev.00/" target="_blank">http://cr.openjdk.java.net/~<u></u>jmasa/7112997/webrev.00/</a><br>
<br>
Thanks.<br>
<br>
</blockquote>
<br>
</blockquote>
</div></div></blockquote></div><br>