On Wed, 14 Jan 2026 21:57:14 GMT, Christopher Schnick <[email protected]> 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
