<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>