On Fri, 19 Jun 2026 01:40:26 GMT, Jaikiran Pai <[email protected]> wrote:

>> Can I please get a review of this change which specifies that the 
>> `IdentityException` throws a `NullPointerException` if the `Class` passed to 
>> its constructor is null?
>> 
>> This change is being proposed as part of the JEP-401 standard library 
>> changes that are currently under review at 
>> https://github.com/openjdk/jdk/pull/31123.
>> 
>> A new jtreg test has been introduced to verify the basic semantics of this 
>> exception class.
>> 
>> ---------
>> - [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 with a new target base due to a 
> merge or a rebase. The incremental webrev excludes the unrelated changes 
> brought in by the merge/rebase. The pull request contains three additional 
> commits since the last revision:
> 
>  - reduce the constructors in IdentityException
>  - merge latest from lworld branch
>  - 8386815: [lworld] IdentityException should specify that it throws 
> NullPointerException if the given Class is null

Good, and maybe change the description of the JBS/PR now as it doesn't match 
where this ended up.

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

Marked as reviewed by alanb (no project role).

PR Review: 
https://git.openjdk.org/valhalla/pull/2562#pullrequestreview-4530292834

Reply via email to