<div>
                    Thanks, John!
                </div>
                <div><div>Here the updated webrev: http://cr.openjdk.java.net/~iveresov/7144296/webrev.01/</div><div><br></div><div>igor</div><div><br></div></div>
                 
                <p style="color: #A0A0A8;">On Friday, February 10, 2012 at 4:50 PM, John Coomes wrote:</p>
                <blockquote type="cite" style="border-left-style:solid;border-width:1px;margin-left:0px;padding-left:10px;">
                    <span><div><div><div>Igor Veresov (<a href="mailto:igor.veresov@oracle.com">igor.veresov@oracle.com</a>) wrote:</div><blockquote type="cite"><div><div>This change does two things: </div><div>1. Prunes the "scavenge roots in code" list after each minor GC.</div><div>2. Promotes objects pointed by code directly into the old gen, instead of ping-ponging them in the survivor spaces.</div><div><br></div><div>Webrev: <a href="http://cr.openjdk.java.net/~iveresov/7144296/webrev.00/">http://cr.openjdk.java.net/~iveresov/7144296/webrev.00/</a></div></div></blockquote><div><br></div><div>Looks good.  Minor suggestion - you could use a typedef and get</div><div>meaningful names for the closures, e.g.,</div><div><br></div><div>typedef PSRootsClosure<true> PSScavengeRootsClosure;</div><div>typedef PSRootsClosure<false> PSPromoteRootsClosure;</div><div><br></div><div>To keep the existing name (PSScavengeRootsClosure) the same at the use</div><div>points, the template class has to be renamed.  I used PSRootsClosure,</div><div>but any other name would work.</div><div><br></div><div>-John</div></div></div></span>
                 
                 
                 
                 
                </blockquote>
                 
                <div>
                    <br>
                </div>