RFR: JDK-8257235: [PATCH] InetAddress.isReachable: Try to use an IPPROTO_ICMP socket type before attempting RAW_SOCK [v2]
Jamie Le Tual
github.com+55101029+jamieletual at openjdk.java.net
Wed Dec 23 14:09:57 UTC 2020
On Tue, 22 Dec 2020 14:08:26 GMT, Alan Bateman <alanb at openjdk.org> wrote:
> Adding support for SOCK_DGRAM/IPPROTO_ICMP(6) is good and I think is mostly okay. The comments and the formatting is messy in several places and should be cleaned up before the patch is sponsored.
Alan, thanks for taking a look at the patch. I tried to see which comments and formatting where messy and I have to say I don't see what your are talking about; as far as I can tell the formatting in the patch doesn't diverge from the code style of the files they are in. Can you give a specific example of what you consider messy so I can know what you're talking about?
-------------
PR: https://git.openjdk.java.net/jdk/pull/1502
More information about the net-dev
mailing list