RFR 8157811 [9] Additional minor fixes and cleanups in NetworkInterface.c

[Langer, Christoph]

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-bounces at openjdk.java.net] On Behalf Of Chris
> Hegarty
> Sent: Mittwoch, 25. Mai 2016 11:50
> To: OpenJDK Network Dev list <net-dev at 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.