On 15/04/2020 16:40, Patrick Concannon wrote:
Hi Alan,
Thanks for your feedback. I've updated the fix with your comments and
you can find them in the new webrev below.
WRT the release note JDK-8237530
<https://bugs.openjdk.java.net/browse/JDK-8237530>, that's a good
point. I will look into removing it.
webrev: http://cr.openjdk.java.net/~pconcannon/8237890/webrevs/webrev.01/
Thanks for the update. Looks good and I've reviewed the CSR too.
Chris is right that we should have tests to check the initial port (and
address) when using the constructors that don't specify them.
-Alan
