RFR(S) 8226645: [TESTBUG] some AppCDS tests relies on illegal reflective access
David Holmes
david.holmes at oracle.com
Wed Aug 14 10:14:09 UTC 2019
+1
This is much clearer to me now.
Thanks,
David
On 14/08/2019 4:32 pm, Alan Bateman wrote:
> On 14/08/2019 01:32, Calvin Cheung wrote:
>> Updated webrev:
>>
>> http://cr.openjdk.java.net/~ccheung/8226645/webrev.02/
>>
>> Summary of changes:
>>
>> - test case provides a lookup instance to Util.defineModifiedClass;
>>
>> - implemented URLClassLoader.loadClass method in LoaderSegregation so
>> no need to use Util.defineClassFromJAR;
>>
>> - removed Util.defineClassFromJAR.
>>
> Can you fix the defineModifiedClass's method description? The comment
> talks about "peerClass" which I think is left over from a previous
> iteration.
>
> In LoaderSegregationTest it might be better to list LoaderSegregation$1
> after its enclosing class.
>
> The rest looks good to me.
>
> -Alan.
>
More information about the hotspot-runtime-dev
mailing list