On 15/03/2018 7:20 PM, Alan Bateman wrote:
On 15/03/2018 08:43, Hamlin Li wrote:
:
Hi Alan,
Thank you for reviewing, I have updated the webrev in place.
( cc'ing serviceability-dev and net-dev as these are the other areas
that use the getFreePort method in the test library. For context, the
patch that we are discussing is:
http://cr.openjdk.java.net/~mli/8199215/webrev.00/ )
The new implementation of getFreePort looks good but it no longer
throws InterruptedException and so might need some of the usages (esp.
in the serviceability tests) to be updated. Also the comment "The
function will spin ..." is no longer relevant and can be removed.
Moving refusingEndpoint() from the NIO test to Utils looks okay. The
"it's much more stable ..." in the method description looks a it
inconsistent with the other wording. An alternative is "This method is
better choice than getFreePort for tests that need an endpoint that
refuses connections".
The update to the tests look okay to me.
Hi Alan,
Thank you for detailed reviewing. I have updated the webrev in place.
(http://cr.openjdk.java.net/~mli/8199215/webrev.00/)
Thank you
-Hamlin
-Alan