> 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: Address review comments ------------- Changes: - all: https://git.openjdk.org/jfx/pull/2032/files - new: https://git.openjdk.org/jfx/pull/2032/files/716fc01c..4ebe1508 Webrevs: - full: https://webrevs.openjdk.org/?repo=jfx&pr=2032&range=08 - incr: https://webrevs.openjdk.org/?repo=jfx&pr=2032&range=07-08 Stats: 3 lines in 2 files changed: 0 ins; 2 del; 1 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
