Hi (Mark or Chris?), as this RFR is outstanding for quite a while and it merely is a minor change which only cleans up coding, would you mind to review it that I can get it off my list?
The only major thing probably is, that calls to getMacAddress don't need a socket anymore and sockets are only opened for platforms where an ioctl is done to determine the MacAddress. As I've said, I've tested it on the Linux/Unix platforms. Thanks a lot Christoph From: Langer, Christoph Sent: Montag, 22. August 2016 15:30 To: 'net-dev@openjdk.java.net' <net-dev@openjdk.java.net> Subject: RE: Ping: RFR(S): 8163181: Further improvements for Unix NetworkInterface native implementation Hi, I made a little update to my change: http://cr.openjdk.java.net/~clanger/webrevs/8163181.2/ Merely comments and a minor AIX specific correction. Thanks in advance for reviewing Christoph From: Langer, Christoph Sent: Dienstag, 16. August 2016 14:49 To: 'net-dev@openjdk.java.net' <net-dev@openjdk.java.net<mailto:net-dev@openjdk.java.net>> Subject: Ping: RFR(S): 8163181: Further improvements for Unix NetworkInterface native implementation Ping: Can I get a review for this small set of changes please? Thanks Christoph From: Langer, Christoph Sent: Donnerstag, 4. August 2016 15:04 To: net-dev@openjdk.java.net<mailto:net-dev@openjdk.java.net> Subject: RFR(S): 8163181: Further improvements for Unix NetworkInterface native implementation Hi, I had made a few more cleanups when I was working on NetworkInterface.c which I thought are worth contributing. Please review: http://cr.openjdk.java.net/~clanger/webrevs/8163181.1/ Bug: https://bugs.openjdk.java.net/browse/JDK-8163181 As always, changes were built and tested on Linux, AIX, Solaris and Mac. Thanks Christoph
