RFR: 8302659: Modernize Windows native code for NetworkInterface [v4]
Daniel Jeliński
djelinski at openjdk.org
Wed Mar 1 18:26:31 UTC 2023
On Wed, 1 Mar 2023 14:37:53 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 one additional commit since the last revision:
>
> Changes as per review comments
Network interface changes LGTM now. I'm not sure about the change to ResolverConfiguration, see below.
src/java.base/windows/native/libnet/ResolverConfigurationImpl.c line 190:
> 188: flags |= GAA_FLAG_SKIP_MULTICAST;
> 189: flags |= GAA_FLAG_SKIP_FRIENDLY_NAME;
> 190: flags |= GAA_FLAG_INCLUDE_ALL_INTERFACES;
Is this needed? As far as I could tell, we only retrieve DNS addresses here, and we only want the DNS servers associated with usable interfaces.
-------------
PR: https://git.openjdk.org/jdk/pull/12593
More information about the build-dev
mailing list