Hi Matthias, looks good to me, too. And +1 for dropping the comment.
Best regards Christoph > -----Original Message----- > From: net-dev <[email protected]> On Behalf Of Chris > Hegarty > Sent: Freitag, 16. August 2019 13:44 > To: Baesken, Matthias <[email protected]> > Cc: [email protected] > Subject: Re: RFR [XS] : 8229706: > java/net/MulticastSocket/NoLoopbackPackets.java fails on some AIX > machines > > > > On 16 Aug 2019, at 10:51, Baesken, Matthias <[email protected]> > wrote: > > > > Thanks alot ! > > > > I guess I can push this as trivial (but a second reviewer would be good > > too > ). > > The change looks fine. > > You could probably drop the comment, I don’t think that > it’s really needed. > > > Best regards, Matthias > > > >> -----Original Message----- > >> From: Daniel Fuchs <[email protected]> > >> ... > >> > >> I haven't observed any failures over 50 runs on all > >> platforms tested. macosx is problem listed for this > >> test so it wasn't run there - but all others platforms > >> passed with your changes. > > The set of checks in the NetworkConfiguration for > has_testableipv6address could probably be expanded to > include macOS. The maybe the test could be removed > from the ProblemList ( for mac ). But that is a separate > issue, for a follow-on, verification, testing, etc. > > -Chris.
