[lworld] RFR: 8282106: [lworld] IdentityException should identify the value class

Tobias Hartmann thartmann at openjdk.org
Fri Jun 28 06:09:32 UTC 2024


On Thu, 27 Jun 2024 20:58:11 GMT, Frederic Parain <fparain at openjdk.org> wrote:

> Improve the message of IdentityException by including the class of the instance that triggered the exception.

What about AArch64? Wouldn't the test now fail on that platform?

test/hotspot/jtreg/runtime/valhalla/inlinetypes/MonitorEnterTest.java line 55:

> 53: 
> 54:   static void monitorEnter(Object o, boolean expectSuccess, String message) {
> 55:     try {

Should be 4-whitespace indentation for Java code.

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

PR Review: https://git.openjdk.org/valhalla/pull/1150#pullrequestreview-2147127579
PR Review Comment: https://git.openjdk.org/valhalla/pull/1150#discussion_r1658198952


More information about the valhalla-dev mailing list