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

Reply via email to