Integrated: 8327651: Rename DictionaryEntry members related to protection domain

Ioi Lam iklam at openjdk.org
Sun Mar 17 06:00:41 UTC 2024


On Tue, 12 Mar 2024 08:18:06 GMT, Ioi Lam <iklam at openjdk.org> wrote:

> 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.

This pull request has now been integrated.

Changeset: d32ce657
Author:    Ioi Lam <iklam at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/d32ce65781c1d7815a69ceac720cdf3ae39caa9e
Stats:     117 lines in 6 files changed: 11 ins; 19 del; 87 mod

8327651: Rename DictionaryEntry members related to protection domain

Reviewed-by: coleenp, dholmes

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

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


More information about the hotspot-runtime-dev mailing list