RFR: 8349145: Make Class.getProtectionDomain() non-native [v3]
Chen Liang
liach at openjdk.org
Mon Feb 3 16:11:07 UTC 2025
On Mon, 3 Feb 2025 14:31:28 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:
>> This change removes the native call and injected field for ProtectionDomain in the java.lang.Class instance, and moves the field to be declared in Java.
>> Tested with tier1-4.
>
> Coleen Phillimore 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 five additional commits since the last revision:
>
> - Hide Class.protectionDomain for reflection and add a test case.
> - Merge branch 'master' into protection-domain
> - Fix two tests.
> - Fix the test.
> - 8349145: Make Class.getProtectionDomain() non-native
The subsequent changes look good.
Something might have changed in the recent bootstrap sequence, but yes it's great to see that Reflection now loads after JLA and can actually use String hash codes.
-------------
Marked as reviewed by liach (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/23396#pullrequestreview-2590379807
PR Comment: https://git.openjdk.org/jdk/pull/23396#issuecomment-2631425455
More information about the core-libs-dev
mailing list