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.

Reply via email to