Review Request JDK-8202113: Reflection API is causing caller classes to leak

Alan Bateman Alan.Bateman at oracle.com
Mon Apr 30 17:21:59 UTC 2018



On 30/04/2018 17:29, mandy chung wrote:
> :
>
>>
>> The 3 x getRoot methods on ReflectAccess looks okay. An alternative 
>> would to create <T extends AccessibleObject> T getRoot(T obj) and a 
>> package private getRoot() method on AccessibleObject.
>>
>
> Good idea.  I updated the patch.
The updated webrev looks good. A minor comment is that I assume you can 
remove the cast from Executable::declaredAnnotations if you leave 
Executable::getRoot in place.

-Alan


More information about the core-libs-dev mailing list