RFR: 8263031: HttpClient throws Exception if it receives a Push Promise that is too large [v2]

[Conor Cleary]

On Mon, 7 Mar 2022 12:31:41 GMT, Daniel Fuchs <dfuchs at openjdk.org> wrote:

>> Conor Cleary has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   8263031: Tidied up import statements
>
> src/java.net.http/share/classes/jdk/internal/net/http/Http2Connection.java line 855:
> 
>> 853: 
>> 854:     private record PushContinuationState(HeaderDecoder pushContDecoder, PushPromiseFrame pushContFrame) {}
>> 855:     private volatile PushContinuationState pcs;
> 
> I'd prefer to have a longer name than `pcs` - `pushContinuationState` would make it less mysterious at the place where it's used.  Also it could be good to move these declarations (linew 854 and 855) higher in the file, where other instance variables are declared.

Changed to pushContinuationState, its a long name but definitely clearer

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

PR: https://git.openjdk.java.net/jdk/pull/7696