On Mon, 20 Jul 2026 10:10:05 GMT, Jayathirth D V <[email protected]> wrote:
> In AreaOp.pruneEdges() we are finalizing and consuming subcurves prematurely > based on link count and this causes artifacts. This PR updates the code to > not finalize the subcurves based on link count. Code is updated to finalize > and consume subcurves only when there is a not connected Y gap. > > Regression tests draws a circle with individual line segments, without the > fix we will see horizontal lines within the circle and with fix we don't see > any artifacts. Code change is also verified with clientlibs CI run. > > --------- > - [x] I confirm that I make this contribution in accordance with the [OpenJDK > Interim AI Policy](https://openjdk.org/legal/ai). test/jdk/java/awt/geom/Area/AreaPath2DDrawTest.java line 28: > 26: * @bug 8386576 > 27: * @summary Checks that when we calculate Area of a path it results > 28: * in proper geometry and no horizontal spurious lines are This proper geometry check is not done..we only check if centre column contains no non-white pixels...It doesn't check that the expected circle outline was drawn, so an all-white/empty result would pass, it seems...I think we should check for positive presence of a known perimeter pixel or validate that the resulting Area contains the expected outline ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/31971#discussion_r3670925726
