On Tue, 21 Jul 2026 01:43:21 GMT, Jaikiran Pai <[email protected]> wrote:

>> 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()

Marked as reviewed by alanb (no project role).

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

PR Review: 
https://git.openjdk.org/valhalla/pull/2649#pullrequestreview-4741769699

Reply via email to