On Wed, 17 Jun 2026 08:26:54 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).

Marked as reviewed by liach (Committer).

test/jdk/java/lang/IdentityExceptionBasicTest.java line 1:

> 1: /*

I thought the jtreg directive block is supposed to appear before imports 
instead of after. Is there a convention on this?

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

PR Review: 
https://git.openjdk.org/valhalla/pull/2562#pullrequestreview-4515946537
PR Review Comment: 
https://git.openjdk.org/valhalla/pull/2562#discussion_r3428426383

Reply via email to