> We can replace `gethostbyname`, which is deprecated on Windows and Linux, > with `getaddrinfo`. This API is available on all supported platforms and so > can be placed in shared code. @djelinski pointed out that `getaddrinfo` can > resolve both IP addresses and host names so the two step approach used in > `networkStream::connect` is not necessary and we can do away with > `os::get_host_by_name()` completely. > > The build is updated to enable winsock deprecation warnings, and now we need > to use `ws2_32.lib` we can drop `wsock32.lib` (as it is basically a subset - > again thanks @djelinski ). > > Testing > - all Oracle builds in tiers 1-5 > - All GHA builds > > The actual code change has to be manually tested because the code is only > used by Ideal Graph Printing to connect to the Ideal Graph Visualizer. I've > manually tested on Windows and Linux and @tobiasholenstein tested macOS. > > Thanks.
David Holmes has updated the pull request incrementally with one additional commit since the last revision: Restrict getaddrinfo to IPv4 only as per the rest of the code ------------- Changes: - all: https://git.openjdk.org/jdk/pull/12842/files - new: https://git.openjdk.org/jdk/pull/12842/files/81f15e05..8b6f6317 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=12842&range=01 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=12842&range=00-01 Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod Patch: https://git.openjdk.org/jdk/pull/12842.diff Fetch: git fetch https://git.openjdk.org/jdk pull/12842/head:pull/12842 PR: https://git.openjdk.org/jdk/pull/12842
