RFR: 8160950 Agent JAR added to app class loader rather than system class loader when running with -Djava.system.class.loader

[serguei.spitsyn at oracle.com]

Thank you for the review, Alan!
Serguei


On 9/8/16 05:59, Alan Bateman wrote:
>
>
> On 07/09/2016 20:27, serguei.spitsyn at oracle.com wrote:
>> Please, review the fix for:
>> https://bugs.openjdk.java.net/browse/JDK-8160950
>>
>> Webrev:
>> http://cr.openjdk.java.net/~sspitsyn/webrevs/2016/jdk/8160950-JLI-WLS.jdk2/ 
>>
>>
> I think the changes are okay.
>
> A simpler custom ClassLoader would be to extend URLClassLoader and 
> have appendToClassPathForInstrumentation invoke addURL. What you have 
> is okay too of course.
>
> -Alan