On Wed, 6 Oct 2021 16:00:26 GMT, Bradford Wetmore <[email protected]> wrote:
>> Andrey Turbanov has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> 8274809: Update java.base classes to use try-with-resources
>> fix review comments
>
> src/java.base/share/classes/sun/security/timestamp/HttpTimestamper.java line
> 115:
>
>> 113:
>> 114: // Send the request
>> 115: try (DataOutputStream output = new
>> DataOutputStream(connection.getOutputStream())) {
>
> Please break this up so it doesn't have lines > 80. e.g.
>
> try (DataOutputStream output =
> new DataOutputStream(connection.getOutputStream())) {
>
> This makes side-by-side viewing very hard without a wider monitor.
>
> Thank you.
Update.
-------------
PR: https://git.openjdk.java.net/jdk/pull/5818
