> This resolves a 25 year old P4 ticket: a NullPointerException is printed to > System.err needlessly. > > This resolution involves confirming that an ImageConsumer is still registered > before every notification. > > I'll understand if this is rejected as unimportant, but I stumbled across > this in the real world the other day and thought this was a simple enough bug > to practice on.
Jeremy has updated the pull request incrementally with one additional commit since the last revision: 4200096: removing whitespace diffs prrace pointed out this branch introduced some pointless whitespace changes. https://github.com/openjdk/jdk/pull/13408#discussion_r1164740594 ------------- Changes: - all: https://git.openjdk.org/jdk/pull/13408/files - new: https://git.openjdk.org/jdk/pull/13408/files/ff469b23..fd751f57 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=13408&range=05 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=13408&range=04-05 Stats: 8 lines in 1 file changed: 0 ins; 0 del; 8 mod Patch: https://git.openjdk.org/jdk/pull/13408.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/13408/head:pull/13408 PR: https://git.openjdk.org/jdk/pull/13408