+1 from me. Looks the same as the FX fix modulo some moving things around.

-phil.

On 7/29/19, 12:56 AM, Laurent Bourgès wrote:
Hi,

Please review this bug fix for the Marlin renderer (introduced in JDK11.0.2):
JBS: https://bugs.openjdk.java.net/browse/JDK-8228711
webrev: http://cr.openjdk.java.net/~lbourges/marlin/marlin-8228711.0/ <http://cr.openjdk.java.net/%7Elbourges/marlin/marlin-8228711.0/>

This patch is very close to MarlinFX patch integrated last week in OpenJFX 14, see https://bugs.openjdk.java.net/browse/JDK-8226789

Changes:
- Stroker: fixed closePath() to preserve last position and its outcode
- TransformingPathConsumer2D: fixed PathClipFilter.closePath() to preserve last position and its outcode
- Dasher: better precision handling (comparison float value with epsilon)
- ClipShapeTest: use preliminary curve subdivision (length > 50px) to avoid false positives on long stroked curves (quad / cubic) + lowered thresholds

Cheers,
Laurent

Reply via email to