<div dir="ltr">And thanks for picking this up and rolling it with yr fix, Thomas!</div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Apr 5, 2017 at 4:38 PM, Srinivas Ramakrishna <span dir="ltr"><<a href="mailto:ysr1729@gmail.com" target="_blank">ysr1729@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><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="HOEnZb"><div class="h5"><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><br>
On Wed, 2017-04-05 at 10:47 -0700, sangheon wrote:<br>
> Hi Thomas,<br>
><br>
</span>[...]<br>
<span class="m_-3847052280932552313im m_-3847052280932552313HOEnZb">> > 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/~ts<wbr>chatzl/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/~ts<wbr>chatzl/8177963/webrev.1</a> (full)<br>
> webrev.1 seems good to me.<br>
<br>
</span><div class="m_-3847052280932552313HOEnZb"><div class="m_-3847052280932552313h5">  thanks for your review.<br>
<br>
Thomas<br>
<br>
</div></div></blockquote></div><br></div>
</div></div></blockquote></div><br></div>