<div dir="ltr"><div>Hi,</div><div><br></div><div>Thanks for the review, Per.</div><div><br></div><div>Removing them sounds good, new webrev:</div><div><a href="https://cr.openjdk.java.net/~manc/8213113/webrev.01/">https://cr.openjdk.java.net/~manc/8213113/webrev.01/</a><br></div><div dir="ltr"><br></div><div dir="ltr">Thanks,<br clear="all"><div><div dir="ltr" class="gmail_signature"><div dir="ltr">Man</div></div></div><br></div></div><br><div class="gmail_quote"><div dir="ltr">On Mon, Nov 5, 2018 at 1:13 AM Per Liden <<a href="mailto:per.liden@oracle.com">per.liden@oracle.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
On 10/31/18 9:51 PM, Man Cao wrote:<br>
> Hi all,<br>
> <br>
> Could anyone review this small clean up? I discovered it while working <br>
> on refactoring code in adaptiveSizePolicy.hpp/cpp.<br>
> Webrev: <a href="https://cr.openjdk.java.net/~manc/8213113/webrev.00/" rel="noreferrer" target="_blank">https://cr.openjdk.java.net/~manc/8213113/webrev.00/</a><br>
> RFE: <a href="https://bugs.openjdk.java.net/browse/JDK-8213113" rel="noreferrer" target="_blank">https://bugs.openjdk.java.net/browse/JDK-8213113</a><br>
<br>
Please also remove the calls to minor_collection_begin() and <br>
minor_collection_end(), otherwise this goes from a trivial "remove dead <br>
code" fix to a change in the actual logic. If these calls are needed at <br>
some point in the future (which I doubt given that CMS is deprecated and <br>
will likely go away soon), they can be added then.<br>
<br>
cheers,<br>
Per<br>
<br>
> Tested on submit repo with the help from JC (CCed).<br>
> <br>
> Thanks,<br>
> Man<br>
</blockquote></div>