RFR: 8375227: Silent OOBE in NGGroup::renderContent [v2]

[Michael Strauß]

On Wed, 14 Jan 2026 21:57:14 GMT, Christopher Schnick <duke at openjdk.org> wrote:

>> This is a very basic solution to the OOBE described in the issue. While it would definitely be better to figure out the root cause, understanding the NGNode implementation is above my knowledge level atm.
>> 
>> The code has been like that for 10 years and no one complained, so I am not sure whether this is a deeper issue or just a small oversight when handling all possible cases.
>> 
>> The fix should retain the exact original workings of the method while avoiding the exception.
>
> Christopher Schnick has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Wrap expression in parentheses

modules/javafx.graphics/src/main/java/com/sun/javafx/sg/prism/NGGroup.java line 259:

> 257:                 NGNode child;
> 258:                 try {
> 259:                     child = orderedChildren.get(i);

You should remove the try-catch block as your change ensures that `List.get()` is never called with an invalid index.

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

PR Review Comment: https://git.openjdk.org/jfx/pull/2037#discussion_r2692494209