On Mon, 12 Jan 2026 18:00:47 GMT, Christopher Schnick <[email protected]> wrote:
>> modules/javafx.graphics/src/main/java/javafx/scene/Scene.java line 5187:
>>
>>> 5185: } else {
>>> 5186: Integer result = slowMap.get(id);
>>> 5187: if (result == null) {
>>
>> I don't think this is an equivalent change
>
> I don't see the issue with this one though. The NPE can only occur if the
> null result is unboxed, right? The get function itself should not throw an
> exception
you are right, `slowMap` is always initialized and a NPE can come only from
unboxing.
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/2032#discussion_r2683339796
