> On May 13, 2016, at 1:20 AM, Sundararajan Athijegannathan > <sundararajan.athijegannat...@oracle.com> wrote: > > Please review http://cr.openjdk.java.net/~sundar/8156914/webrev.00/ for > https://bugs.openjdk.java.net/browse/JDK-8156914 >
This patch looks okay. The renamed TransformerPlugin::transformAndCopy method does not really do the copying wherease the plugin provider has to do the copying. Since you are going to have iterate on it, it’s fine to push this clean up. make/Javadoc.gmk should be updated to only include jdk.tools.jlink.plugin package. Mandy
