<div dir="ltr">Hi Sangheon,<div><br></div><div>The change looks good to me. I'm not an official "Reviewer" though.</div><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>Tue, Apr 30, 2019 at 1:54 PM<br><strong>To: </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 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 each <br>
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>
<br>
Thanks,<br>
Sangheon<br>
</blockquote></div>