RFR: 8175385: ServiceLoader$LazyClassPathLookupIterator scans boot and platform modules for services

[Claes Redestad]

On 02/24/2017 09:50 AM, Alan Bateman wrote:
>
>
> On 23/02/2017 23:19, Claes Redestad wrote:
>> Hi,
>>
>> related to the same regression as JDK-8175561, there are now some cases
>> where we unnecessarily scan platform modules for services.  This patch
>> allows us to skip the platform and boot classloaders in certain cases,
>> which speeds up loading and enumerating the built-in services.
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8175561
>> Webrev: http://cr.openjdk.java.net/~redestad/8175385/jdk.01/
> Looks good.

Thanks!

> A minor nit is to move BootLoader hasClassPath() so it's co-located 
> with the other public static methods.

Moved it up to just after packages() (updated in-place).

/Claes

>
> -Alan