Need reviewers for changes in java/net regression tests
Joe Darcy
joe.darcy at oracle.com
Fri Jul 23 09:45:28 PDT 2010
Hello Pavel.
In principle, I approve this sort of change, but I'd like some
clarifications before granting final approval
Pavel Tisnovsky wrote:
> Hi,
>
> I tried to backport more regression tests from OpenJDK7 to OpenJDK6. Webrev with changes is available at:
> http://cr.openjdk.java.net/~ptisnovs/JavaNetJTregTests/
>
It would be helpful to know what JDK 7 changesets these fixes came from
and what the differences from the JDK 7 changes are in OpenJDK 6.
> Can anyone please review the changes?
>
> There are still minor differences between OpenJDK6 and OpenJDK7 tests, mainly due to syntax enhancements in OpenJDK7.
>
> Please note that new test case (deserialization an Inet6 address with a named interface) is included in Inet6Address/serialize/Serialize.java regression test:
> http://cr.openjdk.java.net/~ptisnovs/JavaNetJTregTests/test/java/net/Inet6Address/serialize/Serialize.java.cdiff.html
>
> This issue which was fixed in OpenJDK7 but not in OpenJDK6 yet. Btw - do anyone know, if this issue will be fixed in OpenJDK6?
>
Generally, the @bug list in the test should indicate what bugs the test
addresses. Is there a changeset for bug 6656849? If so, you could
request approval to backport it.
Regards,
-Joe
> The changes made in these test are really minor and they are based (but
> not equal) on changes done by Christopher Hegarthy in OpenJDK7:
> http://hg.openjdk.java.net/jdk7/tl/jdk/rev/58f325ba3e27
> (these changes are reviewed by Alan Bateman)
>
> Cheers
> Pavel Tisnovsky
>
More information about the jdk6-dev
mailing list