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
