Code Review 6934054: java/net/Socket/FDClose.java return error in samevm
Alan.Bateman at Sun.COM
Thu Mar 11 07:42:25 PST 2010
Christopher Hegarty -Sun Microsystems Ireland wrote:
> Hi Alan, Michael,
> Another small test change!
> This test requires interruptible I/O to verify that the accept
> implementation does not impact on fd0. UseVMInterruptibleIO needs to
> be explicitly enabled since it is now disabled by default, see
> 6554406. Without interruptible I/O the thread executing the accept
> gets blocked and the test harness cannot perform the necessary cleanup
> required by samevm.
> Note: This test is only applicable to Solaris, since it is the only
> platform to support interruptible I/O, but should still pass on all
I'm tempted to suggest that the best thing is to just hg rm the test as
it's not very useful. If you want to keep it then the changes looks OK
to me. One suggestion is to have AReader implement Closeable and have
the close implement just do sock.close().
More information about the net-dev