Michael McMahon wrote:
Christopher Hegarty -Sun Microsystems Ireland wrote:
Hi Michael,
The following tests have incorrect assertion failures if run with
assertions enabled:
test/java/net/CookieHandler/TestHttpCookie.java
Would it be better to just remove the assertion in the first one as well?
I don't think it really matters. The assertion just verifies that the
that the construction is not called with null, which seems reasonable.
It is just this one specific test that doesn't conform since it verifies
that NPE is actually thrown. If you feel strongly I can remove it.
-Chris.
test/java/net/URLClassLoader/closetest/CloseTest.java
Webrev:
http://cr.openjdk.java.net/~chegar/6935199/webrev.00/webrev/
-Chris.
