> Can I please get a review of this change which reorders a null check in > `ExplodedImage`? While at it, the `isClosable` is made `final` in > `JrtFileSystem` too. > > > > > --------- > - [x] I confirm that I make this contribution in accordance with the [OpenJDK > Interim AI Policy](https://openjdk.org/legal/ai).
Jaikiran Pai has updated the pull request incrementally with one additional commit since the last revision: Alan's review - let NPE be descriptive, remove usage of Objects.requireNonNull() ------------- Changes: - all: https://git.openjdk.org/valhalla/pull/2649/files - new: https://git.openjdk.org/valhalla/pull/2649/files/23ea5709..14ab9780 Webrevs: - full: https://webrevs.openjdk.org/?repo=valhalla&pr=2649&range=01 - incr: https://webrevs.openjdk.org/?repo=valhalla&pr=2649&range=00-01 Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod Patch: https://git.openjdk.org/valhalla/pull/2649.diff Fetch: git fetch https://git.openjdk.org/valhalla.git pull/2649/head:pull/2649 PR: https://git.openjdk.org/valhalla/pull/2649
