Re: RFR: 8334772: Change Class::signers to an explicit field [v3]

2024-07-18 Thread David Holmes
On Thu, 18 Jul 2024 13:48:06 GMT, Chen Liang 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

Re: RFR: 8334772: Change Class::signers to an explicit field [v3]

2024-07-18 Thread Chen Liang
On Thu, 18 Jul 2024 13:48:06 GMT, Chen Liang 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

Re: RFR: 8334772: Change Class::signers to an explicit field [v3]

2024-07-18 Thread David Holmes
On Thu, 18 Jul 2024 13:48:06 GMT, Chen Liang 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

Re: RFR: 8334772: Change Class::signers to an explicit field [v3]

2024-07-18 Thread Roger Riggs
On Thu, 18 Jul 2024 13:48:06 GMT, Chen Liang 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

Re: RFR: 8334772: Change Class::signers to an explicit field [v3]

2024-07-18 Thread Chen Liang
> `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.) > > Mi

Re: RFR: 8334772: Change Class::signers to an explicit field [v2]

2024-07-18 Thread Alan Bateman
On Thu, 18 Jul 2024 13:31:02 GMT, Chen Liang 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

Re: RFR: 8334772: Change Class::signers to an explicit field [v2]

2024-07-18 Thread Chen Liang
> `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.) > > Mi

Re: RFR: 8334772: Change Class::signers to an explicit field

2024-07-18 Thread Coleen Phillimore
On Wed, 17 Jul 2024 19:47:44 GMT, Chen Liang 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 Sec

Re: RFR: 8334772: Change Class::signers to an explicit field

2024-07-18 Thread Chen Liang
On Wed, 17 Jul 2024 19:47:44 GMT, Chen Liang 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 Sec

Re: RFR: 8334772: Change Class::signers to an explicit field

2024-07-18 Thread Alan Bateman
On Thu, 18 Jul 2024 12:30:24 GMT, Coleen Phillimore wrote: > 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 HPROF heap dump has a slot for signers so have to keep that to avoid breakage. So yes, it mea

Re: RFR: 8334772: Change Class::signers to an explicit field

2024-07-18 Thread Coleen Phillimore
On Wed, 17 Jul 2024 19:47:44 GMT, Chen Liang 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 Sec

Re: RFR: 8334772: Change Class::signers to an explicit field

2024-07-18 Thread Alan Bateman
On Wed, 17 Jul 2024 19:47:44 GMT, Chen Liang 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 Sec

Re: RFR: 8334772: Change Class::signers to an explicit field

2024-07-17 Thread David Holmes
On Wed, 17 Jul 2024 19:47:44 GMT, Chen Liang 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 Sec

RFR: 8334772: Change Class::signers to an explicit field

2024-07-17 Thread Chen Liang
`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 JN