RFR: 8343957: Rename ObjectMonitor::owner_from() and JavaThread::_lock_id [v2]
Patricio Chilano Mateo
pchilanomate at openjdk.org
Wed Dec 4 15:47:26 UTC 2024
> Please review this small renaming patch. During the review of JDK-8338383 there were some comments about improving the naming for `ObjectMonitor::owner_from()` and `JavaThread::_lock_id`. These originate from the changes introduced to inflated monitors, where we now record the `java.lang.Thread.tid` of the owner in the ObjectMonitor's `_owner` field instead of a `JavaThread*`. I renamed `_lock_id` as `_monitor_owner_id` and `owner_from()` as `owner_id_from()`.
>
> Thanks,
> Patricio
Patricio Chilano Mateo has updated the pull request incrementally with one additional commit since the last revision:
Fix parameter name
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/22524/files
- new: https://git.openjdk.org/jdk/pull/22524/files/108abc2b..8cfc9a10
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=22524&range=01
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=22524&range=00-01
Stats: 3 lines in 1 file changed: 0 ins; 0 del; 3 mod
Patch: https://git.openjdk.org/jdk/pull/22524.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/22524/head:pull/22524
PR: https://git.openjdk.org/jdk/pull/22524
More information about the graal-dev
mailing list