On Wed, 5 Feb 2025 17:57:29 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 incrementally with three 
> additional commits since the last revision:
> 
>  - Update 
> test/jdk/java/lang/reflect/AccessibleObject/TrySetAccessibleTest.java
>    
>    Co-authored-by: David Holmes 
> <62092539+dholmes-...@users.noreply.github.com>
>  - Update 
> test/jdk/java/lang/reflect/AccessibleObject/TrySetAccessibleTest.java
>    
>    Co-authored-by: David Holmes 
> <62092539+dholmes-...@users.noreply.github.com>
>  - Remove @Stable annotation for final field.

The core library changes look good. Need some other reviewer to verify the VM 
changes.

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

Marked as reviewed by liach (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/23396#pullrequestreview-2596620084

Reply via email to