RFR: 8289385: Cleanup redundant synchronization in Http2ClientImpl [v2]

Daniel Fuchs dfuchs at openjdk.org
Thu Jun 30 07:49:41 UTC 2022


On Wed, 29 Jun 2022 20:57:14 GMT, Andrey Turbanov <aturbanov at openjdk.org> wrote:

>> Http2ClientImpl.stopping and Http2ClientImpl.failures are always accessed under synchronized(this).
>> So we can remove 'volatile' modifier from 'stopping'. And remove 'synchronizedSet' wrapper from 'failures'
>
> Andrey Turbanov has updated the pull request incrementally with one additional commit since the last revision:
> 
>   8289385: Cleanup redundant synchronization in Http2ClientImpl
>   add comments

Marked as reviewed by dfuchs (Reviewer).

-------------

PR: https://git.openjdk.org/jdk/pull/9311


More information about the net-dev mailing list