On Fri, 16 Jan 2026 18:58:45 GMT, Kevin Rushforth <[email protected]> wrote:
>> This should improve the code quality of the class while preserving its
>> original workings
>
> 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
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/2043#discussion_r2699670793
