RFR: 8327651: Rename DictionaryEntry members related to protection domain [v3]

Ioi Lam iklam at openjdk.org
Sat Mar 16 22:33:37 UTC 2024


> Rename `DictionaryEntry::_pd_set` to `_package_access_cache`, and rename all related methods accordingly.
> 
> The old name was very confusing. It makes one wonder,
> 
>> Is `DictionaryEntry::_pd_set` related to the `java.lang.Class.getProtectionDomain()` of the `DictionaryEntry::_instance_klass` in any way?
> 
> (Hint, they have absolutely no relationship to each other).
> 
> I am making this PR as future changes in the Leyden project may be touching this area. I don't want the reviewers of my future PRs be confused by this code.
> 
> ----------------
> This PR is mostly name changes and comment clarification.
> 
> Other than a log message change for clarification, there are no behavioral changes in this PR.

Ioi Lam has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains three additional commits since the last revision:

 - Merge branch 'master' of https://github.com/openjdk/jdk into 8327651-Rename-DictionaryEntry-members-for-Protection-Domain
 - @coleenp review -- add comments about avoiding Java upcalls
 - 8327651: Rename DictionaryEntry members related to protection domain

-------------

Changes:
  - all: https://git.openjdk.org/jdk/pull/18222/files
  - new: https://git.openjdk.org/jdk/pull/18222/files/b0ca86d1..4a7e3737

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=18222&range=02
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=18222&range=01-02

  Stats: 18912 lines in 431 files changed: 10232 ins; 6922 del; 1758 mod
  Patch: https://git.openjdk.org/jdk/pull/18222.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/18222/head:pull/18222

PR: https://git.openjdk.org/jdk/pull/18222


More information about the hotspot-runtime-dev mailing list