<div dir="ltr"><div><div><div><div><div>Jim,<br><br></div>Here is a final webrev (for archive):<br><a href="http://cr.openjdk.java.net/~lbourges/marlin/marlin-8144938.5/">http://cr.openjdk.java.net/~lbourges/marlin/marlin-8144938.5/</a><br><br></div>Changes:<br></div>- remove loops in CrashNaNTest<br><br></div>Hope it is ready to go !<br><br></div>I need another reviewer, please.<br><br><div><div><div><div><div><div><div><div class="gmail_extra"><br><div class="gmail_quote">2016-03-18 1:44 GMT+01:00 Jim Graham <span dir="ltr"><<a href="mailto:james.graham@oracle.com" target="_blank">james.graham@oracle.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Laurent,<br>
<br>
That looks fine.  If the subPathStarted changes are the only changes then it's good to go.  If there was some other change that I missed, let me know so I can review it as well.<br></blockquote><div><br></div><div>No this webrev had only changes in MRE.pathToLoop() you reviewed.<br> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
One curiosity - in the test case, why are there 3 for loops that perform exactly one pass?  (lines 134, 221, 282)  I'm guessing that maybe in a former life this code may have been part of a performance suite or something?  It's not a bug, just an odd bit of code.  You can delete them if you want, but I don't need to see a review of that...<br></blockquote><div><br></div><div>Fixed, now.<br><br></div><div>Cheers,<br></div><div>Laurent</div></div></div></div></div></div></div></div></div></div></div>