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
Tue Feb 16 13:33:41 UTC 2021
On Mon, 18 Jan 2021 08:25:34 GMT, Alan Bateman <alanb at openjdk.org> wrote:
>> Jamie Le Tual has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Fixed formatting
>
> src/java.base/unix/native/libnet/Inet6AddressImpl.c line 713:
>
>> 711: This usually requires "root" privileges, so it's likely to fail.
>> 712: If all else fails, fall back to TCP and implement tcp echo
>> 713: */
>
> This is one of the block comments that needs a tidy, same thing in Inet4Address.c. Also check the // comments and you'll see some of the inconsistencies there. It's just nit picking, the patch itself is good, just hard to test.
OK, is it the fact that it's a block comment instead of a line comment? Is the indentation not right?
I don't see what needs to be cleaned up
-------------
PR: https://git.openjdk.java.net/jdk/pull/1502
More information about the net-dev
mailing list