Hi Chris (or anyone who is holding a stake in the NetworkInterface native implementation),
I've spent some time on cleaning up NetworkInterface.c and came up with a bigger change: http://cr.openjdk.java.net/~clanger/webrevs/8160174.2/ With this I attempted to consolidate the interface listing functionality of the 2 main categories - one is ioctl (used on Linux IPv4, Solaris and AIX) and the other is getifaddrs (used on MacOS). I introduced some defines to switch between the implementations. I also consolidated the functionality for the ioctl based network interface listings by using an #ifdef section to distinguish between the Linux/AIX versus Solaris field and constant names. I've spent some time testing on the platforms and in principal it works. But as it is a matter of taste, I'd like to ask you if you support this type of change or have any hints or recommendations before going forward to finalize this. For Linux I'm also suggesting to use the getifaddrs approach - I tested and found it working everywhere and with this we could get rid of the implementation to read /proc/net for IPv6. Furthermore I'm generally setting Null for the IPv6 broadcast address - which I think is common sense for IPv6. Thanks in advance and best regards Christoph From: Langer, Christoph Sent: Donnerstag, 23. Juni 2016 16:37 To: 'net-dev@openjdk.java.net' <net-dev@openjdk.java.net> Subject: RFR (S) JDK-8160174: java.net.NetworkInterface - fixes and improvements for network interface listing Hi, can I please get a review the following change: Webrev: http://cr.openjdk.java.net/~clanger/webrevs/8160174.1/ Bug: https://bugs.openjdk.java.net/browse/JDK-8160174 I made further fixes and cleanups for listing Unix type network interfaces, especially on AIX and Linux. I ran builds and verified also on Solaris and Mac. Thanks Christoph
