On Mon, 12 Jan 2026 17:22:59 GMT, Andy Goryachev <[email protected]> wrote:
>> Christopher Schnick has updated the pull request incrementally with two
>> additional commits since the last revision:
>>
>> - Fix visibility
>> - Fixes
>
> 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
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/2032#discussion_r2683319759
