<div dir="ltr"><div dir="ltr">Renaming sounds good to me too, no need for another webrev.<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" class="gmail_attr"><strong>From: </strong><span dir="ltr"><<a href="mailto:sangheon.kim@oracle.com">sangheon.kim@oracle.com</a>></span><br><strong>Date: </strong>Mon, May 6, 2019 at 10:08 AM<br><strong>To: </strong>Thomas Schatzl<br><strong>Cc: </strong>hotspot-gc-dev<br><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Thomas,<br>
<br>
On 5/6/19 3:12 AM, Thomas Schatzl wrote:<br>
> Hi,<br>
><br>
> On Tue, 2019-04-30 at 13:54 -0700, <a href="mailto:sangheon.kim@oracle.com" target="_blank">sangheon.kim@oracle.com</a> wrote:<br>
>> Hi all,<br>
>><br>
>> Can I have some reviews that replace assert(used() ==<br>
>> recalculated_used()) with better message?<br>
>> Those assert messages are not much helpful as it doesn't include any<br>
>> actual values on it so I'm suggesting to include actual values of<br>
>> each and then replace as a macro.<br>
>><br>
>> CR: <a href="https://bugs.openjdk.java.net/browse/JDK-8221509" rel="noreferrer" target="_blank">https://bugs.openjdk.java.net/browse/JDK-8221509</a><br>
>> webrev: <a href="http://cr.openjdk.java.net/~sangheki/8221509/webrev.0/" rel="noreferrer" target="_blank">http://cr.openjdk.java.net/~sangheki/8221509/webrev.0/</a><br>
>> testing: local build<br>
>    thanks for fixing this :)<br>
><br>
> The only nit I have is maybe the assert could include what is actually<br>
> checked, ie. something like assert_used_and_recalculate_used_equal()<br>
> instead of just assert_used_and_recalculate_used().<br>
><br>
> I would not need a re-review for the renaming.<br>
Thanks for the review and the naming suggestion.<br>
I will rename for 'assert_used_and_recalculate_used_equal()' before the <br>
push.<br>
<br>
Thanks,<br>
Sangheon<br>
<br>
<br>
><br>
> Thanks,<br>
>    Thomas<br>
><br>
<br>
</blockquote></div></div>