> 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 with a new target base due to a merge or a rebase. The pull request now contains nine commits: - Address review comments - Add test clarifying generics handling - Fix failing tests - Improve log messages and fix more catch blocks in SelectBinding - Fix visibility - Fixes - Fixes - More fixes - Handle exceptions better ------------- Changes: https://git.openjdk.org/jfx/pull/2032/files Webrev: https://webrevs.openjdk.org/?repo=jfx&pr=2032&range=07 Stats: 430 lines in 9 files changed: 36 ins; 229 del; 165 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
