Sorry I had that test change already. I've filed another bug to track your suggestions.
Please review http://cr.openjdk.java.net/~sundar/8160346/webrev.00/ for https://bugs.openjdk.java.net/browse/JDK-8160346 Thanks, -Sundar On 6/25/2016 4:10 AM, Mandy Chung wrote: >> On Jun 24, 2016, at 9:54 AM, Sundararajan Athijegannathan >> <sundararajan.athijegannat...@oracle.com> wrote: >> >> Please review http://cr.openjdk.java.net/~sundar/8153238/webrev.00/ for >> https://bugs.openjdk.java.net/browse/JDK-8153238 > 86 if (builtInPlugins.size() < numPlugins) { > > What about just count the plugins defined in jdk.jlink module (i.e. matching > what getNumJlinkPlugins counts)? > > Since you are on this file, -verbose:gc is not needed for a test. Maybe good > to take this out. > > 56 * @run main/othervm -verbose:gc -Xmx1g JLinkTest > > Mandy