RFR 8160346: JLinkTest.java should compute exact number of plugins from jdk.jlink module [ was Re: RFR 8153238: Improve test/tools/jlink/JLinkTest.java not to hard code the number of plugins]

[Jim Laskey (Oracle)]

+1

> On Jun 27, 2016, at 8:19 AM, Sundararajan Athijegannathan <sundararajan.athijegannathan at oracle.com> wrote:
> 
> 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.athijegannathan at 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
>