On 15/10/2013 21:58, Brian Burkhalter wrote:
Pursuant to comments on the submitted CCC request, the patch previously
approved for this issue has been revised:
http://cr.openjdk.java.net/~bpb/7179567/webrev.6/
The difference is that the URLClassLoader constructors and newInstance()
methods no longer throw a NPE if the URL array parameter is non-null but
contains at least one null element. The javadoc and test have been adjusted
accordingly.
A new issue https://bugs.openjdk.java.net/browse/JDK-8026517 has been filed to
track adding a test for a null element in the URL array parameter. This latter
issue is not presently scheduled to be addressed in JDK 8.
Thanks,
Brian
This version looks okay too. For JDK-8026517 then you could add a note
to say that the tests for it are in the NullURLTest.java test case,
alternatively remove the commented-out tests and leave it to when
JDK-8026517 is re-examined.
-Alan.
