RFR: 8327651: Rename DictionaryEntry members related to protection domain [v2]
Coleen Phillimore
coleenp at openjdk.org
Wed Mar 13 12:22:16 UTC 2024
On Tue, 12 Mar 2024 22:26:40 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.
>
> Ioi Lam has updated the pull request incrementally with one additional commit since the last revision:
>
> @coleenp review -- add comments about avoiding Java upcalls
Ok, looks good.
-------------
Marked as reviewed by coleenp (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/18222#pullrequestreview-1933975568
More information about the hotspot-runtime-dev
mailing list