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
Fri Nov 29 09:11:09 UTC 2024
On Thu, 28 Nov 2024 16:26:55 GMT, Daniel Fuchs <dfuchs at openjdk.org> wrote:
>> For IntelliJ's `'SocketChannel' used without 'try'-with-resources statement` warning on `open()` method. Am I supposed to not suppress IDE warnings?
>
> Oh - OK - I see. That's fine. We usually don't suppress warnings in tests. Maybe we should. On the other hand `@SuppressWarnings` is a two edged sword as it can hide other issues if its scope is too large.
👍 Switched to a more fine-grained suppression.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/22160#discussion_r1863192025
More information about the net-dev
mailing list