On Mon, 20 Jul 2026 16:32:17 GMT, Xin Liu <[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).
>
> src/java.desktop/share/classes/sun/awt/geom/AreaOp.java line 419:
> 
>> 417:             // intermediate subcurves to optimize memory footprint
>> 418:             if (linkCount >= MAX_LINK_COUNT) {
>> 419:                 consumeSubCurves(subcurves, chains, ret);
> 
> If we revert this change, do we mitigate JDK-8350035? (I can't access it) 
> or this was just an optimization of memory?

JDK-8350035 is completely different bug and doesn't touch AreaOp code.

This change in the original bug was only a part of memory optimization.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/31971#discussion_r3619157259

Reply via email to