RFR: 8247972: incorrect implementation of JVM TI GetObjectMonitorUsage [v21]
Serguei Spitsyn
sspitsyn at openjdk.org
Mon Mar 4 11:01:46 UTC 2024
On Mon, 4 Mar 2024 03:28:28 GMT, David Holmes <dholmes at openjdk.org> wrote:
>> Serguei Spitsyn has updated the pull request incrementally with one additional commit since the last revision:
>>
>> rename after merge: jvmti_common.h to jvmti_common.hpp
>
> test/hotspot/jtreg/serviceability/jvmti/ObjectMonitorUsage/ObjectMonitorUsage.java line 40:
>
>> 38: * - owned object with N waitings to be notified
>> 39: * - owned object with N waitings to enter, from 0 to N waitings to re-enter,
>> 40: * from N to 0 waitings to be notified
>
> "waitings" is not grammatically valid. Suggestion: s/waitings/threads waiting/
Thanks, fixed now.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/17680#discussion_r1510976777
More information about the hotspot-dev
mailing list