RFR: 8375016: Several catch blocks for NullPointerExceptions exist in the codebase [v2]

[Christopher Schnick]

On Mon, 12 Jan 2026 17:22:59 GMT, Andy Goryachev <angorya at openjdk.org> 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