Re: Review Request: JDK-8165346 j.l.ClassLoader.getDefinedPackage(String) throws NPE

2016-09-09 Thread Alan Bateman
On 08/09/2016 23:02, Mandy Chung wrote: Spec bug: missing @throws NPE if the specified name is null in the relevant getPackage methods: ClassLoader.getDefinedPackage, ClassLoader::getPackage, Package::getPackage Webrev at: http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8165346/webrev.00

Re: Review Request: JDK-8165346 j.l.ClassLoader.getDefinedPackage(String) throws NPE

2016-09-08 Thread Aleksey Shipilev
On 09/09/2016 01:02 AM, Mandy Chung wrote: > Spec bug: missing @throws NPE if the specified name is null in the relevant > getPackage methods: > ClassLoader.getDefinedPackage, ClassLoader::getPackage, Package::getPackage > > Webrev at: > http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8165346

Re: Review Request: JDK-8165346 j.l.ClassLoader.getDefinedPackage(String) throws NPE

2016-09-08 Thread Lance Andersen
+1 > On Sep 8, 2016, at 6:02 PM, Mandy Chung wrote: > > Spec bug: missing @throws NPE if the specified name is null in the relevant > getPackage methods: > ClassLoader.getDefinedPackage, ClassLoader::getPackage, Package::getPackage > > Webrev at: > http://cr.openjdk.java.net/~mchung/jdk9/webr

Review Request: JDK-8165346 j.l.ClassLoader.getDefinedPackage(String) throws NPE

2016-09-08 Thread Mandy Chung
Spec bug: missing @throws NPE if the specified name is null in the relevant getPackage methods: ClassLoader.getDefinedPackage, ClassLoader::getPackage, Package::getPackage Webrev at: http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8165346/webrev.00/index.html Mandy