RFR: 8332610: Remove confusing comment in ObjectMonitor::ReenterI
Coleen Phillimore
coleenp at openjdk.org
Tue May 21 14:27:01 UTC 2024
On Tue, 21 May 2024 13:14:37 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.
Please remove that unused variable also in the two places it's defined. There's a perf counter that does not use this variable so the comment above with TryLock does make sense wrt to that perf counter. This local variable is unused.
-------------
Changes requested by coleenp (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/19326#pullrequestreview-2068799531
More information about the hotspot-runtime-dev
mailing list