Hi Arthur,

This looks good to me. I have launched the changes in your second
webrev through our test system and not observed any regression.
I don't know if Chris still has some tests running, but from
my standpoint you're good to go!

best regards,

-- daniel

On 11/03/2019 18:14, Arthur Eubanks wrote:
Updated copyright years (I asked around, it should be fine), updated commit message.
http://cr.openjdk.java.net/~aeubanks/8220083/webrev.01

On Mon, Mar 11, 2019 at 10:46 AM Martin Buchholz <marti...@google.com <mailto:marti...@google.com>> wrote:

    These changes are clean progress and allow tests to pass on any of
    the 4 (!) combinations of IPV4/IPV6 support available.  LGTM.

    BUT ... in the current multi-protocol world it's probably better to
    test against all the interfaces available on the host machine.
    Maybe there should be a InetAddress.getLoopbackAddresses (note the
    plural!) and tests should run against all the InetAddress objects
    returned by such an API?
    But this is a big task (mostly spec work I would guess) for the
    net-dev team.

    On Mon, Mar 11, 2019 at 9:55 AM Arthur Eubanks <aeuba...@google.com
    <mailto:aeuba...@google.com>> wrote:

        bug: https://bugs.openjdk.java.net/browse/JDK-8220083
        webrev: http://cr.openjdk.java.net/~aeubanks/8220083/webrev.00/

        First of a few ipv6 patches to come.


Reply via email to