RFR: 8334772: Change Class::signers to an explicit field
Coleen Phillimore
coleenp at openjdk.org
Thu Jul 18 12:32:31 UTC 2024
On Wed, 17 Jul 2024 19:47:44 GMT, Chen Liang <liach at openjdk.org> wrote:
> `Class` has 2 VM-injected fields that can be made explicit: `Object[] signers` and `ProtectionDomain protectionDomain`. We make the signers field explicit. (The ProtectionDomain can be revisited when SecurityManager is removed, as SecurityManager is accessing it via JNI as well.)
>
> Migrate the JNI code to Java. The getter previously had a redundant primitive type check, which is dropped in the migrated Java code. The `Object[] getSigners` is no longer `native`, thus requiring a CSR record. Reviewers please help review the associated CSR.
I thought we moved this already. There's a change in the heapDumper.cpp that probably has to change also, because I think we're now dumping signers twice (two lines). The one in jvmtiTagMap.cpp reports the SIGNERS tag so that has to stay.
-------------
PR Review: https://git.openjdk.org/jdk/pull/20223#pullrequestreview-2185676651
More information about the core-libs-dev
mailing list