Reviewer and committer request for 7198496 (original) (raw)

Peter Levart peter.levart at gmail.com
Wed Oct 3 12:41:44 UTC 2012

I think that in the LazyIterator.hasNext() you still have to check for null loader:

337 if (loader == null) 338 configs = ClassLoader.getSystemResources(fullName); 339 else 340 configs = loader.getResources(fullName);

Because the ClassLoader.getSystemClassLoader() can return null. In that case, ClassLoader.getSystemResources() delegates just to .getBootstrapResources() which is in sync with later:

359 S p = service.cast(Class.forName(cn, true, loader) 360 .newInstance());

when loader is null - i.e. it loads just bootstrap classes.

Regards, Peter

On 10/03/2012 01:03 PM, Paul Sandoz wrote:

Hi,

See here: http://cr.openjdk.java.net/~psandoz/tl/7198496/webrev/ Thanks, Paul.

More information about the core-libs-dev mailing list