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

Kevin Rushforth kcr at openjdk.org
Wed Jan 14 22:07:48 UTC 2026


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

I'm not sure this is equivalent. Also, I'd like to see a test for this. In general, adding null checks or other similar checks without understanding them is not something we want to do.

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

Changes requested by kcr (Lead).

PR Review: https://git.openjdk.org/jfx/pull/2037#pullrequestreview-3663032530


More information about the openjfx-dev mailing list