I missed that as well.  Good catch.

> On Sep 6, 2016, at 11:50 AM, Anthony Vanelverdinghe 
> <anthony.vanelverdin...@gmail.com> wrote:
> 
> Hi Sundar
> 
> I'm not familiar with the code, but the for-loop at lines 105-107 looks like 
> a bug, since only the first element is considered:
> 
> 105             for (String jvmlib : jvmlibs) {
> 106                 return t.path().endsWith("/" + jvmlib);
> 107             }
> 
> Kind regards,
> Anthony
> 
> 
> On 6/09/2016 12:12, Sundararajan Athijegannathan wrote:
>> Please review http://cr.openjdk.java.net/~sundar/8163952/webrev.01/ for
>> https://bugs.openjdk.java.net/browse/JDK-8163952
>> 
>> Thanks,
>> 
>> -Sundar
>> 
> 

Reply via email to