RFR(XS): 8163973: VM Anonymous classes should not call Class File Load Hooks
Rachel Protacio
rachel.protacio at oracle.com
Tue Aug 16 19:08:03 UTC 2016
Thanks for the review, Lois!
As discussed offline, this change won't need an added test.
Rachel
On 8/16/2016 7:47 AM, Lois Foltan wrote:
> Hi Rachel,
>
> I think this looks good.
>
> You might consider adding a nested anonymous class test for this. I
> believe Harold has developed one for JDK-8030221. I think it would be
> good to make sure that the host_klass parameter is indeed NULL for a
> nested anonymous class and that the VM does not go forward into the
> CFLH code in that scenario.
>
> Thanks,
> Lois
>
> On 8/15/2016 4:48 PM, Rachel Protacio wrote:
>> Hello,
>>
>> Please review this change, which makes sure class file load hooks are
>> not called for VM anonymous classes. See
>> http://mail.openjdk.java.net/pipermail/core-libs-dev/2016-January/038353.html
>> for justification.
>>
>> Passes JPRT and RBT.
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8163973
>> Open webrev: http://cr.openjdk.java.net/~rprotacio/8163973.00/
>>
>> Thank you!
>> Rachel
>
More information about the hotspot-runtime-dev
mailing list