On Tue, 3 Dec 2024 19:10:55 GMT, Patricio Chilano Mateo <pchilanom...@openjdk.org> wrote:
> 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 Renaming looks good and makes it clearer what the id is. ------------- Marked as reviewed by coleenp (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/22524#pullrequestreview-2476917190
