> 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

Reply via email to