RFR: 8289385: Cleanup redundant synchronization in Http2ClientImpl [v2]
Andrey Turbanov
aturbanov at openjdk.org
Wed Jun 29 20:57:15 UTC 2022
On Wed, 29 Jun 2022 15:04:46 GMT, Daniel Fuchs <dfuchs at openjdk.org> wrote:
>> Andrey Turbanov has updated the pull request incrementally with one additional commit since the last revision:
>>
>> 8289385: Cleanup redundant synchronization in Http2ClientImpl
>> add comments
>
> src/java.net.http/share/classes/jdk/internal/net/http/Http2ClientImpl.java line 59:
>
>> 57:
>> 58: private final HttpClientImpl client;
>> 59: private boolean stopping;
>
> before this line and before line 68 - could you please add a comment that says:
>
> // only accessed from within synchronized blocks
>
>
> Otherwise it looks reasonable to me.
Done
-------------
PR: https://git.openjdk.org/jdk/pull/9311
More information about the net-dev
mailing list