<div dir="ltr">Thank you for the code review!<div><br clear="all"><div><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">-Man</div></div></div><br></div></div><br><div class="gmail_quote"><div dir="ltr">On Wed, Nov 7, 2018 at 11:12 PM 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"><br>
On 2018-11-05 23:28, Man Cao wrote:<br>
> Hi,<br>
> <br>
> Thanks for the review, Per.<br>
> <br>
> Removing them sounds good, new webrev:<br>
> <a href="https://cr.openjdk.java.net/~manc/8213113/webrev.01/" rel="noreferrer" target="_blank">https://cr.openjdk.java.net/~manc/8213113/webrev.01/</a> <br>
> <<a href="https://cr.openjdk.java.net/%7Emanc/8213113/webrev.01/" rel="noreferrer" target="_blank">https://cr.openjdk.java.net/%7Emanc/8213113/webrev.01/</a>><br>
<br>
Looks good.<br>
<br>
cheers,<br>
Per<br>
<br>
> <br>
> Thanks,<br>
> Man<br>
> <br>
> <br>
> On Mon, Nov 5, 2018 at 1:13 AM Per Liden <<a href="mailto:per.liden@oracle.com" target="_blank">per.liden@oracle.com</a> <br>
> <mailto:<a href="mailto:per.liden@oracle.com" target="_blank">per.liden@oracle.com</a>>> wrote:<br>
> <br>
>     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<br>
>     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>
>     <<a href="https://cr.openjdk.java.net/%7Emanc/8213113/webrev.00/" rel="noreferrer" target="_blank">https://cr.openjdk.java.net/%7Emanc/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<br>
>     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>
> <br>
</blockquote></div>