[lworld] RFR: 8287241: Add IdentityException to report that a value object is not valid [v3]

Mandy Chung mchung at openjdk.java.net
Thu Jun 9 18:17:28 UTC 2022


On Thu, 9 Jun 2022 15:53:02 GMT, Rémi Forax <forax at openjdk.org> wrote:

>> It not clear where to draw the line on convenience methods (message can be null), but ok, added.
>
> It's a convenient method but it's not equivalent to calling IdentityException(null, cause), Throwable(cause) is equivalent to Throwable(cause.getMessage(), cause) which is important if later a logger is configured to only log the error message.

Exceptions in the JDK do not always have all four standard constructors as in `Throwable`.  There was a discussion to audit the constructor choices for JDK exceptions.  Some new exceptions do add all 4 constructors (`IllegalCallerException` and `WrongThreadException`).   Since this one has a constructor taking the `cause` for exception chaining, it's reasonable to add the variant setting the default message to `cause.getMessage()`.

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

PR: https://git.openjdk.org/valhalla/pull/696



More information about the valhalla-dev mailing list