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

Kevin Rushforth kcr at openjdk.org
Wed Jan 14 22:47:58 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

OK, looking at it more closely, I now see why it is equivalent. A comment would probably help. And let us know what you come up with as far as a test goes.

Thanks.

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

PR Comment: https://git.openjdk.org/jfx/pull/2037#issuecomment-3752036351


More information about the openjfx-dev mailing list