RFR: 8233296: MulticastSocket getOption/setOption inverts the value of IP_MULTICAST_LOOP
Chris Hegarty
chris.hegarty at oracle.com
Thu Nov 21 16:47:45 UTC 2019
> On 21 Nov 2019, at 16:46, Daniel Fuchs <daniel.fuchs at oracle.com> wrote:
>
> On 21/11/2019 15:02, Chris Hegarty wrote:
>> The source changes look good.
>> A few minor comments on the test.
>> You probably don’t need this anymore? ( InetAddress.getLoopbackAddress() does not throw )
>> 63 SetLoopbackOption() throws Exception {
>> 64 }
>> You could also push the main entry point to the bottom, out of the way. I guess it is just for quickly running locally.
>
> Done.
>
> http://cr.openjdk.java.net/~dfuchs/webrev_8233296/webrev.01/
I was kinda suggesting that the explicit no-args constructor is not needed ( but maybe I have missed something? ). Either way ( I’ll leave it to you ), you have my Review.
-Chris.
More information about the net-dev
mailing list