Thank you for the review, Alan!
Serguei

On 9/8/16 05:59, Alan Bateman wrote:


On 07/09/2016 20:27, [email protected] 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


Reply via email to