Reviewer and committer request for 7197642

Alan Bateman Alan.Bateman at oracle.com
Mon Oct 1 17:35:14 UTC 2012


On 01/10/2012 14:14, Paul Sandoz wrote:
> Hi,
>
> See here:
>
>    http://cr.openjdk.java.net/~psandoz/tl/7197642/webrev/
>
> Thanks,
> Paul.
If I have it right, this "moves" the NPE from the ServiceLoader's 
iterator methods to the load* methods where the check should have been. 
It's a subtle behavior change but I think it's okay for a major release.

The test looks good except that it's partying like it's 1999 :-)  I'm 
happy to sponsor this and I can fix up the copyright date before pushing 
the change. I'll leave it until tomorrow to give other a chance to jump 
in if they wish.

-Alan.



More information about the core-libs-dev mailing list