> 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 two 
additional commits since the last revision:

 - Fix visibility
 - Fixes

-------------

Changes:
  - all: https://git.openjdk.org/jfx/pull/2032/files
  - new: https://git.openjdk.org/jfx/pull/2032/files/855ed2f5..9465dc83

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jfx&pr=2032&range=01
 - incr: https://webrevs.openjdk.org/?repo=jfx&pr=2032&range=00-01

  Stats: 49 lines in 1 file changed: 10 ins; 10 del; 29 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

Reply via email to