RFR: 8343791: Socket.connect API should document whether the socket will be closed when hostname resolution fails or another error occurs [v22]
Volkan Yazıcı
duke at openjdk.org
Thu Nov 28 15:53:41 UTC 2024
On Thu, 28 Nov 2024 15:22:39 GMT, Daniel Fuchs <dfuchs at openjdk.org> wrote:
>> Volkan Yazıcı has updated the pull request incrementally with two additional commits since the last revision:
>>
>> - Don't exceed 120 characters per line
>> - Rework functional interfaces in tests
>
> test/jdk/java/net/Socket/ConnectFailTest.java line 144:
>
>> 142: }
>> 143:
>> 144: @SuppressWarnings("resource")
>
> What is this SuppressWarnings for?
For IntelliJ's `'SocketChannel' used without 'try'-with-resources statement` warning on `open()` method. Am I supposed to not suppress IDE warnings?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/22160#discussion_r1862418705
More information about the net-dev
mailing list