7026376: (dc) DatagramChannel created without specifying protocol family fails to join IPv4 group
Alan Bateman
Alan.Bateman at oracle.com
Tue Mar 15 02:41:05 PDT 2011
Chris Hegarty wrote:
> :
>
> I'm fine with this change as is, but here are a few minor comments:
>
> In DatagramChannelImpl.innerJoin() the
> else { throw new IllegalArgumentException("Address type not
> supported"); }
> seems unnecessary, but maybe ok if we expect support for new protocol
> families to be added DatagramChannelImpl.
You're right, this is really a "should not reach here" path and should
be an AssertionError. I'll fix that.
>
> I guess canJoin6WithIPv4Group and canIPv6SocketJoinIPv4Group don't
> have to be implemented as native methods, but they look fine.
I toyed with an additional class to encapsulate the platform
capabilities but it would require a class per platform and hardly seems
worth it for this.
-Alan
More information about the nio-dev
mailing list