On Mon, 3 Feb 2025 14:31:28 GMT, Coleen Phillimore <cole...@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