RFR: 8302659: Modernize Windows native code for NetworkInterface [v3]

Alan Bateman alanb at openjdk.org
Mon Feb 27 15:29:28 UTC 2023


On Wed, 22 Feb 2023 17:40:31 GMT, Rich DiCroce <duke at openjdk.org> wrote:

>> Improves performance and correctness, as discussed on the net-dev mailing list.
>
> Rich DiCroce has updated the pull request incrementally with two additional commits since the last revision:
> 
>  - Forgot to add file
>  - Resolve review comments

src/java.base/windows/native/libnet/NetworkInterface.c line 288:

> 286: }
> 287: 
> 288: static jobject createNetworkInterfaceForSingleRowWithTables(

I think it would be helpful for future maintainers if there was a comment on each of static functions to make it clear what they do.

src/java.base/windows/native/libnet/NetworkInterface.h line 39:

> 37:     struct _netaddr *Next;
> 38: } netaddr;
> 39: 

Can NetworkInterface.h be dropped, and netaddr moved to NetworkInterface.c?

-------------

PR: https://git.openjdk.org/jdk/pull/12593



More information about the build-dev mailing list