On Fri, 19 Jun 2026 16:04:22 GMT, ExE Boss <[email protected]> wrote: >> 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 > > src/java.base/share/classes/java/lang/IdentityException.java line 74: > >> 72: * that the cause is nonexistent or unknown. >> 73: */ >> 74: public IdentityException(Throwable cause) { > > I’d rather we kept these, as it’s annoying to have to use > [`Throwable::initCause(…)`]. > > [`Throwable::initCause(…)`]: > https://docs.oracle.com/en/java/javase/26/docs/api/java.base/java/lang/Throwable.html#initCause(java.lang.Throwable)
Hello @ExE-Boss, these can be reintroduced at some point, if there's a genuine need for them. For new APIs, it's better to start off small, including for exception types. ------------- PR Review Comment: https://git.openjdk.org/valhalla/pull/2562#discussion_r3443550973
