<div dir="ltr">+1<br><div><br></div><div>..Thomas</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Jan 17, 2019 at 10:32 AM Aleksey Shipilev <<a href="mailto:shade@redhat.com">shade@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 1/16/19 6:21 PM, Aleksey Shipilev wrote:<br>
> RFE:<br>
>   <a href="https://bugs.openjdk.java.net/browse/JDK-8217014" rel="noreferrer" target="_blank">https://bugs.openjdk.java.net/browse/JDK-8217014</a><br>
> <br>
> Fix:<br>
>   <a href="http://cr.openjdk.java.net/~shade/8217014/webrev.01/" rel="noreferrer" target="_blank">http://cr.openjdk.java.net/~shade/8217014/webrev.01/</a><br>
<br>
Got an external comment: since we are changing the end-of-run message, it makes sense be to able to<br>
print sizes in K, not in M. The simple addendum is to use proper_unit infra for heap/metadata<br>
occupancy messages.<br>
<br>
Hopefully final webrev:<br>
  <a href="http://cr.openjdk.java.net/~shade/8217014/webrev.03/" rel="noreferrer" target="_blank">http://cr.openjdk.java.net/~shade/8217014/webrev.03/</a><br>
<br>
Thanks,<br>
-Aleksey<br>
<br>
</blockquote></div>