Thanks. I included it, though it may not show in the webrev ( white space change )
I generalise the issue to cover a few other minor issues. Webrev updated in-place: http://cr.openjdk.java.net/~chegar/8157811/ -Chris. > On 25 May 2016, at 13:59, Langer, Christoph <christoph.lan...@sap.com> wrote: > > Hi Chris, > > looks nice, I had seen some of these places, too. > > Here is another one which you could add: > > --- a/src/java.base/unix/native/libnet/NetworkInterface.c Tue May 24 > 10:14:41 2016 -0700 > +++ b/src/java.base/unix/native/libnet/NetworkInterface.c Wed May 25 > 14:56:31 2016 +0200 > @@ -1829,7 +1829,7 @@ > strncpy(if2.lifr_name, ifname, sizeof(if2.lifr_name) - 1); > > if (ioctl(sock, SIOCGLIFFLAGS, (char *)&if2) < 0) { > - return -1; > + return -1; > } > > *flags = if2.lifr_flags; > > Best regards > Christoph > >> -----Original Message----- >> From: net-dev [mailto:net-dev-boun...@openjdk.java.net] On Behalf Of Chris >> Hegarty >> Sent: Mittwoch, 25. Mai 2016 11:50 >> To: OpenJDK Network Dev list <net-dev@openjdk.java.net> >> Subject: RFR 8157811 [9] Additional minor fixes and cleanups in >> NetworkInterface.c >> >> As a follow up to JDK-8156521, and when comparing against the >> version of this file in jdk8u-dev a few minor issues were noticed. >> >> There is a free of a memory structure that was missed somehow >> in the 9 version of this file ( it is already in the 8 version ). The >> remaining few changes are just some minor cleanup. >> >> http://cr.openjdk.java.net/~chegar/8157811/ >> https://bugs.openjdk.java.net/browse/JDK-8157811 >> >> -Chris.