<div dir="auto"><div>Hi Sergey,<div dir="auto"><br></div><div dir="auto">Thanks for your comment.<br><div dir="auto"><br><div dir="auto">This new test only validates the new clipping algorithms ie it compares the rendering outputs with / without clipping enabled.</div><div dir="auto"><br></div><div dir="auto">As such algorithms are only available in Marlin 0.8.2 and the test uses new system properties to enable/disable clipping, I confirm it passes before (jdk9 or jdk10 before patch).</div><div dir="auto"><br></div><div dir="auto">To ensure it detects any regression, I manually inserted some bugs in the clipping code, and the test failed.</div><div dir="auto"><br></div><div dir="auto">Note: I should add another test @run to check the float variant too (and not only the double variant, the default in jdk10).</div><div dir="auto"><br></div><div dir="auto">Finally I could write a new performance test that would prove clipping is more efficient than before.</div><div dir="auto">Such test would fail before patch (timeout ?), but it is difficult to make it robust as it depends on the hw. </div><div dir="auto">Jim wrote a basic test in the jfx bug showing 300ms without but 2ms now => gain is high. </div><div dir="auto">A possible success condition would be: clipping gain > 10 or 50.</div><div dir="auto"><br></div><div dir="auto">Regards,</div><div dir="auto">Laurent</div></div></div><br><div class="gmail_extra"><br><div class="gmail_quote">Le 4 déc. 2017 11:11 PM, "Sergey Bylokhov" <<a href="mailto:Sergey.Bylokhov@oracle.com">Sergey.Bylokhov@oracle.com</a>> a écrit :<br type="attribution"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi, Laurent.<div class="quoted-text"><br>
On 29/11/2017 14:30, Laurent Bourgès wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
- added new ClipShapeTest (jtreg) that checks all possible combinations of (cap / join) for random polyline (Stroker) and polygons (Filler) comparing image outputs rendered with clipping enabled vs disabled<br>
</blockquote>
<br></div>
I have only one note that the test is passed before the fix, so if we will regress at some point later we will not catch this.<font color="#888888"><br>
<br>
<br>
-- <br>
Best regards, Sergey.<br>
</font></blockquote></div><br></div></div></div>