On 17/02/2016 11:43, Chris Hegarty wrote:
I know this has gone through several rounds of review already.
I do not want to stall progress, I am reasonably happy with the
changes, but I have a few comments ( that can be addressed later,
if needed ).
- MulticastSocket.java: I would add a javadoc link to
{@linkplain StandardSocketOptions.SO_REUSEPORT SO_REUSEPORT}
I've had to adjust a few things in the patch, this happens to me one of
them so that it links to the description. I'll post a patch before this
is pushed to make sure that Lucy and everyone is okay with it.
-Alan.