RFR: 8289385: Cleanup redundant synchronization in Http2ClientImpl [v2]
kristylee88
duke at openjdk.org
Sat Jul 2 18:43:52 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 kristylee88 at github.com (no known OpenJDK username).
-------------
PR: https://git.openjdk.org/jdk/pull/9311
More information about the net-dev
mailing list