Reviewer and committer request for 7198496

[Peter Levart]

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.