<div dir="ltr"><div><div><div><div><div><div><div><div>Sergey,<br><br></div>Please review this simple fix to the Dasher problem:<br></div>JBS: <a href="https://bugs.openjdk.java.net/browse/JDK-8202580">https://bugs.openjdk.java.net/browse/JDK-8202580</a><br></div>webrev: <a href="http://cr.openjdk.java.net/~lbourges/marlin/marlin-8202580.0/">http://cr.openjdk.java.net/~lbourges/marlin/marlin-8202580.0/</a><br><br></div>Changes:<br></div>- (D)Dasher.init: the given dash array is dirty as MarlinRenderingEngine got it from XxxArrayCache (Unsafe.allocateUninitializedArray) when calling copyDashArray(). The fix consists in using the correct part [0; dashLen[ as it was the case in jdk9<br></div>- added test that detects wrong dashed rectangle: jtreg fails on jdk10/11 without patch<br></div><div><br></div><div>Notes: <br></div><div>- the thread┬ámay hang in init()  if the dirty part contains negative values (normalization loop never exits), but it is very difficult to reproduce and test.</div><div></div><div><div><div><div><div><div><div><div><div></div></div></div></div></div></div></div></div>- My Apologies: I introduced the bug in dec 2016 when I merged MarlinFX / Marlin2D.<br></div></div><div></div><div><br></div><div><br></div><div>Finally, the proposed fix should be backported to JDK10 and also to OpenJFX 10/11. I can propose a patch to OpenJFX11 soon.</div><div><br></div><div>Could somebody else manage the backport process (JDK-update 10, OpenJFX10) ?<br></div><div><br></div><div>Regards,</div><div>Laurent<br></div></div>