On 08/09/11 21:40, Alan Bateman wrote:
Michael McMahon wrote:
Sigh. Hopefully this is the last webrev.

http://cr.openjdk.java.net/~michaelm/7085981/webrev.4/

Socket.java changed from last one. If there's no objection
I'll push this version.

- Michael.
DatagramSocket(SocketAddress) creates an unbound socket if the parameter is null. If I read the changes correctly then it will close it for this case. I think the try/finally needs to be narrowed to just the bind. Same thing in MulticastSocket.

-Alan.


Yes, the regression tests picked that up as well. Well spotted.

http://cr.openjdk.java.net/~michaelm/7085981/webrev.5/

Thanks
Michael.




Reply via email to