<div dir="ltr">Re webrev.1: Good, but I think you shouldn't print the overflow message each time overflow occurs while writing, but only at the end when the timestamp entries are printed. That should suffice to let user know an overflow occurred and how big a size they may need (that shd be part of the overflow message).<div><br></div><div>-- ramki</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Apr 5, 2017 at 11:16 AM, Thomas Schatzl <span dir="ltr"><<a href="mailto:thomas.schatzl@oracle.com" target="_blank">thomas.schatzl@oracle.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Sangheon,<br>
<span class=""><br>
On Wed, 2017-04-05 at 10:47 -0700, sangheon wrote:<br>
> Hi Thomas,<br>
><br>
</span>[...]<br>
<span class="im HOEnZb">> > New webrevs at:<br>
> > <a href="http://cr.openjdk.java.net/~tschatzl/8177963/webrev.0_to_1" rel="noreferrer" target="_blank">http://cr.openjdk.java.net/~<wbr>tschatzl/8177963/webrev.0_to_1</a> (diff)<br>
> > <a href="http://cr.openjdk.java.net/~tschatzl/8177963/webrev.1" rel="noreferrer" target="_blank">http://cr.openjdk.java.net/~<wbr>tschatzl/8177963/webrev.1</a> (full)<br>
> webrev.1 seems good to me.<br>
<br>
</span><div class="HOEnZb"><div class="h5">  thanks for your review.<br>
<br>
Thomas<br>
<br>
</div></div></blockquote></div><br></div>