RFR: 8332610: Remove unused nWakeups ObjectMonitor [v3]

Daniel D. Daugherty dcubed at openjdk.org
Tue May 21 16:37:01 UTC 2024


On Tue, 21 May 2024 14:48:15 GMT, Johan Sjölen <jsjolen at openjdk.org> wrote:

>> Hi,
>> 
>> This comment only serves to confuse people, as `nWakeups` is not shared between threads and never read. It is probably meant for debugging (not sure). Let's delete the comment.
>
> Johan Sjölen has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Delete the second nWakeups

Thumbs up. This is a trivial fix.

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

Marked as reviewed by dcubed (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/19326#pullrequestreview-2069117219


More information about the hotspot-runtime-dev mailing list