Re: Code Review 6935199: java/net regression tests failing with Assertions

2010-03-16 Thread Christopher Hegarty -Sun Microsystems Ireland
Michael McMahon wrote: Christopher Hegarty -Sun Microsystems Ireland wrote: 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/TestHtt

Re: Code Review 6935199: java/net regression tests failing with Assertions

2010-03-16 Thread Michael McMahon
Christopher Hegarty -Sun Microsystems Ireland wrote: 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 b

Re: Code Review 6935199: java/net regression tests failing with Assertions

2010-03-16 Thread Christopher Hegarty -Sun Microsystems Ireland
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 a

Re: Code Review 6935199: java/net regression tests failing with Assertions

2010-03-16 Thread Michael McMahon
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? test/java/net/

Code Review 6935199: java/net regression tests failing with Assertions

2010-03-16 Thread Christopher Hegarty -Sun Microsystems Ireland
Hi Michael, The following tests have incorrect assertion failures if run with assertions enabled: test/java/net/CookieHandler/TestHttpCookie.java test/java/net/URLClassLoader/closetest/CloseTest.java Webrev: http://cr.openjdk.java.net/~chegar/6935199/webrev.00/webrev/ -Chris.