7026376: (dc) DatagramChannel created without specifying protocol family fails to join IPv4 group

[Alan Bateman]

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