<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
</head>
<body text="#000000" bgcolor="#FFFFFF">
Approved.<br>
<br>
-phil.<br>
<br>
<div class="moz-cite-prefix">On 9/6/19 2:15 PM, Laurent Bourgès
wrote:<br>
</div>
<blockquote type="cite"
cite="mid:CAKjRUT4vdZaAAKQzEBM2g10EWZgSapRSZn=eW6=pnDeuVb6CuA@mail.gmail.com">
<meta http-equiv="content-type" content="text/html; charset=UTF-8">
<div dir="ltr">
<div>Hi,</div>
<div><br>
</div>
<div>Please review this bug fix for the Marlin renderer (present
in Pisces code since JDK 6?):</div>
<div>JBS: <a
href="https://bugs.openjdk.java.net/browse/JDK-8230728"
moz-do-not-send="true">https://bugs.openjdk.java.net/browse/JDK-8230728</a></div>
<div>webrev: <a
href="http://cr.openjdk.java.net/~lbourges/marlin/marlin-8230728.0/"
moz-do-not-send="true">http://cr.openjdk.java.net/~lbourges/marlin/marlin-8230728.0/</a></div>
<div><br>
</div>
<div>This patch fixes a NaN handling in userSpaceLineWidth()
when the affine transform determinant is negative (flip bit is
ON)</div>
<div><br>
</div>
<div>Changes:</div>
<div>- MarlinR<span style="color:rgb(0,0,0)">enderingEngine: use
abs(at.getDeterminant()) to ensure positive value before
using sqrt</span></div>
<div><span style="color:rgb(0,0,0)">- FlipBitTest: new jtreg
test reproducing the original problem drawing a thin ellipse
with a flipped transform<br>
</span></div>
<div><br>
<span style="color:rgb(0,0,0)"></span></div>
<div><span style="color:rgb(0,0,0)">Cheers,</span></div>
<div><span style="color:rgb(0,0,0)"></span>Laurent</div>
</div>
</blockquote>
<br>
</body>
</html>