RFR[8241072]: 'Reimplement Legacy DatagramSocket API'
Chris Hegarty
chris.hegarty at oracle.com
Tue Mar 31 17:27:16 UTC 2020
> On 30 Mar 2020, at 19:27, Patrick Concannon <patrick.concannon at oracle.com> wrote:
> ...
>
> http://cr.openjdk.java.net/~pconcannon/8241072/webrevs/webrev.01/
This looks very good, a testament to all the prior cleanup work that has already been done.
Just a few comments:
- DatagramSocket::getChannel now stands out a little, since it is one of the few instance methods has still has a non-delegating body. It could optionally delegate, or if not, then NetMulticastSocket doesn’t necessarily need to override it.
- In DatagramSocket::createDelegate, "enable broadcast if possible” - Possibly due to refactoring, but I cannot reconcile this with the old implementation.
- The createDelegate's @return comment is a bit hard to parse; suggest: "@return {@code null} if {@code bindaddr == NO_DELEGATE}, OTHERWISE returns a delegate for the requested {@code type}"
- The set of socket options is now per datagram/multicast socket instance, rather than on the class of the socket. I don’t think that this is an issue, just an observation and a confirmation that it is deliberate.
-Chris.
More information about the net-dev
mailing list