On Fri, 16 Jan 2026 19:11:54 GMT, Christopher Schnick <[email protected]> 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

Reply via email to