<div dir="ltr"><div><div><div>Jim & Phil,<br><br></div>Here is the updated webrev:<br><a href="http://cr.openjdk.java.net/~lbourges/marlin/marlin-8149338.1/">http://cr.openjdk.java.net/~lbourges/marlin/marlin-8149338.1/</a><br><br></div><div>I just simplified the exception handling code in both TextClipErrorTest and CrashNaNTest</div></div><div><div><div><br></div><div>Cheers,<br></div><div>Laurent<br></div><div><div class="gmail_extra"><br><div class="gmail_quote">2016-02-10 0:46 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">Let me know if/when you have an updated webrev.  It should be good to go, but it couldn't hurt to publish one for the archives...<br>
<br>
                        ...jim<span class=""><br>
<br>
On 2/9/16 12:51 AM, Laurent Bourgès wrote:<br>
</span><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="">
Jim,<br>
<br>
Here are my answers to your 2 questions:<br>
<br>
2016-02-09 0:14 GMT+01:00 Jim Graham <<a href="mailto:james.graham@oracle.com" target="_blank">james.graham@oracle.com</a><br></span>
<mailto:<a href="mailto:james.graham@oracle.com" target="_blank">james.graham@oracle.com</a>>>:<span class=""><br>
<br>
    In the test case, why are you using a log handler to check for a<br>
    particular exception?  Shouldn't any exception logged be cause for a<br>
    test failure?<br>
<br>
<br>
I already used that code in 1 other test: TextClipErrorTest. I agree it<br>
can be simpler to report any exception like:<br>
             public void publish(LogRecord record) {<br>
                 Throwable th = record.getThrown();<br>
                 // detect any Throwable:<br>
                 if (th != null) {<br>
                     System.out.println("Test failed:\n" +<br>
record.getMessage());<br>
                     th.printStackTrace(System.out);<br>
<br>
                     throw new RuntimeException("Test failed: ", th);<br>
                 }<br>
             }<br>
<br>
    Is there a reason why you reversed the calculations for the slope at<br>
    line 374?<br>
<br>
+ final double slope = (x1d - x2) / (y1d - y2);<br>
<br>
I prefer this syntax as it is more explicit that (x1d - x2) and (y1d -<br>
y2) are double values (not implicit promotion).<br>
<br>
Cheers,<br>
Laurent<br>
</span></blockquote>
</blockquote></div><br></div></div></div></div></div>