<div dir="ltr"><br><div>I had a look at webrev.2 below, and the changes look good to me.</div><div><br></div><div>Reviewed!</div><div>- ramki  (openjdk: ysr)</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Mar 7, 2016 at 4:03 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,<br>
<span class=""><br>
On Fri, 2016-03-04 at 17:52 +0100, Thomas Schatzl wrote:<br>
> Hi,<br>
><br>
> On Fri, 2016-03-04 at 11:14 -0500, Tony Printezis wrote:<br>
> > Thomas,<br>
> ><br>
> > I confirmed that the code below was incorrect and doing the<br>
> > addition<br>
> > before calling restore() fixes the issue. Latest webrev here:<br>
> ><br>
> > <a href="http://cr.openjdk.java.net/~tonyp/8146991/webrev.2/" rel="noreferrer" target="_blank">http://cr.openjdk.java.net/~tonyp/8146991/webrev.2/</a><br>
> ><br>
><br>
>   looks good to me. JPRT has been fine, although I messed up running<br>
> the other tests - I forgot to specify ParallelGC as collector to use.<br>
><br>
> I will redo that on Monday. I do not expect any issues.<br>
<br>
</span>Looks all good. I will wait one or two days to push for any potential<br>
additional reviews.<br>
<br>
Thanks,<br>
  Thomas<br>
<br>
</blockquote></div><br></div>