On Mon, 27 Nov 2023 02:09:56 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/serviceability/jvmti/GetOwnedMonitorInfo/GetOwnedMonitorInfoTest.java
> line 58:
>
>> 56: // Inject this situation into this test that performs other
>> 57: // GetOwnedMonitorInfo testing.
>> 58: Object obj = new Object() { public String toString() {return
>> "";} };
>
> Nit: the `toString` definition is not needed. This could just be `new
> Object();`, or `new Object() {};` if you want to introduce a nested class.
Thanks. Done.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/16783#discussion_r1405891539
