RFR: 8264999: GeneralPath.lineTo() to itself produces jagged lines

Laurent Bourgès lbourges at openjdk.java.net
Wed Jun 8 17:47:40 UTC 2022


On Mon, 30 May 2022 01:15:01 GMT, Jeremy <duke at openjdk.java.net> wrote:

> Previously when you drew a line that moved exactly (+0, +0) with a miter stroke the Stroker class acted like a horizontal miter was appropriate. Now we abort instead, so no miter is applied.
> 
> The new unit test tests the original steps plus several variations.
> 
> I'm a little nervous about this change because it looks like someone previously considered this exact condition and implemented a different behavior. I don't think I understand the original rationale.

As mentioned in my code review, the stroker changes need one more condition: (prev == drawTo) return.

I tested this change and it fixes the bug's test code and your test code looks good.

Please update PR with the proposed change and we are good.

Deadline: tomorrow, sorry.

-------------

PR: https://git.openjdk.java.net/jdk/pull/8943



More information about the client-libs-dev mailing list