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