RFR: 8326568: jdk/test/com/sun/net/httpserver/bugs/B6431193.java should use try-with-resource and try-finally [v2]
Daniel Fuchs
dfuchs at openjdk.org
Tue Apr 2 13:30:13 UTC 2024
On Wed, 27 Mar 2024 15:11:40 GMT, Daniel Fuchs <dfuchs at openjdk.org> wrote:
>> Darragh Clarke has updated the pull request incrementally with one additional commit since the last revision:
>>
>> implemented feedback
>
> test/jdk/com/sun/net/httpserver/bugs/B6431193.java line 60:
>
>> 58: t.sendResponseHeaders(200, response.length());
>> 59: os.write(response.getBytes());
>> 60: os.close();
>
> no need to call `os.close()` here since it will be called by the `try ( ) { }` block.
line with `os.close()` should be removed
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/18514#discussion_r1547882399
More information about the net-dev
mailing list