Reviewer and committer request for 7197642 (original) (raw)
Paul Sandoz paul.sandoz at oracle.com
Tue Oct 2 07:34:12 UTC 2012
- Previous message: Reviewer and committer request for 7197642
- Next message: ServiceLoader.load(Class, ClassLoader) bug
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
On Oct 1, 2012, at 7:35 PM, Alan Bateman <Alan.Bateman at oracle.com> wrote:
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.
Yes, more specifically the first call to Iterator.getNext(), and an somewhat obfuscated NPE would have been thrown.
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.
Thanks, Paul.
- Previous message: Reviewer and committer request for 7197642
- Next message: ServiceLoader.load(Class, ClassLoader) bug
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]