Chris Hegarty wrote:
Looks fine to me. It might be nicer to have "boolean connected", which is set to true when connectToAddress succeeds.Alan, Michael,connect(SocketAddress,int) should close the underlying file descriptor if it fails for any reason. This is consistent with the other connect methods. A little other cleanup while there.http://cr.openjdk.java.net/~chegar/6994079/webrev.00/webrev/ -Chris.
-Alan