RFR: 8375561: Class NGGroup is in need of some cleanup
Christopher Schnick
duke at openjdk.org
Fri Jan 16 20:52:07 UTC 2026
On Fri, 16 Jan 2026 19:11:54 GMT, Christopher Schnick <duke at openjdk.org> wrote:
>> modules/javafx.graphics/src/main/java/com/sun/javafx/sg/prism/NGGroup.java line 221:
>>
>>> 219: List<NGNode> orderedChildren = getOrderedChildren();
>>> 220: for (NGNode orderedChild : orderedChildren) {
>>> 221: orderedChild.renderForcedContent(gOptional);
>>
>> If `getOrderedChildren()` can ever return null, then the null check is needed, so I can't tell without digging a little whether this is a safe fix.
>
> Should I add the explanations in the PR or the JBS issue? Andy also asked for a summary in the JBS issue, so ideally I would combine that
Added a detailed summary for the changes in the JBS issue
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/2043#discussion_r2699922383
More information about the openjfx-dev
mailing list