On Wed, 6 Oct 2021 16:00:26 GMT, Bradford Wetmore <wetm...@openjdk.org> 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

Reply via email to