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