RFR: 8375016: Several catch blocks for NullPointerExceptions exist in the codebase [v4]
Christopher Schnick
duke at openjdk.org
Thu Jan 15 17:47:46 UTC 2026
> This PR replaces various catch blocks for NPEs by proper null checks. It looks like a lot but most of the binding changes are just variants of the same approach. The test cases have been adapted to not check for NPEs anymore.
>
> The PrismFontFactory change to a RuntimeException was made because I don't see how the try block can throw an NPE.
Christopher Schnick has updated the pull request incrementally with one additional commit since the last revision:
Improve log messages and fix more catch blocks in SelectBinding
-------------
Changes:
- all: https://git.openjdk.org/jfx/pull/2032/files
- new: https://git.openjdk.org/jfx/pull/2032/files/70377bde..6da3fd16
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jfx&pr=2032&range=03
- incr: https://webrevs.openjdk.org/?repo=jfx&pr=2032&range=02-03
Stats: 20 lines in 1 file changed: 3 ins; 3 del; 14 mod
Patch: https://git.openjdk.org/jfx/pull/2032.diff
Fetch: git fetch https://git.openjdk.org/jfx.git pull/2032/head:pull/2032
PR: https://git.openjdk.org/jfx/pull/2032
More information about the openjfx-dev
mailing list