Re: RFR: 8320515: assert(monitor->object_peek() != nullptr) failed: Owned monitors should not have a dead object [v5]

[David Holmes]

On Mon, 27 Nov 2023 01:45:09 GMT, David Holmes <dhol...@openjdk.org> wrote:

>> Stefan Karlsson has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   Split test and use othervm
>
> test/hotspot/jtreg/runtime/Monitor/MonitorWithDeadObjectTest.java line 75:
> 
>> 73:     private static void testDetachThread() {
>> 74:         // Create an ObjectMonitor with a dead object from an
>> 75:         // attached thread.
> 
> Unclear what the "Detach" in the method name has to do with anything. ??

And why add these wrapper methods that simply call one other method?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/16783#discussion_r1405546829