Christopher Hegarty - Sun Microsystems Ireland wrote:
Hi Alan, Michael,
Trivial change.
6933618: java/net/MulticastSocket/NoLoopbackPackets.java fails when rerun
If java/net/MulticastSocket/NoLoopbackPackets.java is run in a batch
which contains all the MulticastSocket tests then it will fail on
reruns with java.lang.NoSuchMethodError: Sender.<init>.
The reason it fails is that there is another test,
java/net/MulticastSocket/SetOutgoingIf.java, that has a class named
Sender. This test is typically run after NoLoopbackPackets so its
Sender class gets left over in the classes directory and is used in
subsequest runs of NoLoopbackPackets. The Sender class in
NoLoopbackPackets is not compatible with the Sender class in
SetOutgoingIf.
NoLoopbackPackets should define Sender as an internal class avoiding
this name conflict. Also remove hardcoded port number.
Webrev:
http://cr.openjdk.java.net/~chegar/6933618/webrev.00/
Thanks,
-Chris.
Looks okay to me - one small suggestion is to close msock in a finally
block so that the file descriptor usage doesn't creep up during these
samevm runs.
-Alan.