Thanks for looking at this, Alan and Chris. Please check the updated
webrev:
http://cr.openjdk.java.net/~xiaofeya/8042581/webrev.01/
-Felix
On 12/29/2014 11:51 PM, Alan Bateman wrote:
On 29/12/2014 15:46, Chris Hegarty wrote:
Looks ok Felix.
You could put the construction of the new DatagramSocket in a
try-with-resources, so it will be closed always, even if the test fails.
-Chris
I agree with Chris' comment. One other thing is that testing if e
instanceof BindException is not necessary as e will always be a
BindException.
-Alan
