On Wed, 4 Jun 2025 17:43:16 GMT, Daniel Fuchs <dfu...@openjdk.org> wrote:

>> Daniel Fuchs has updated the pull request with a new target base due to a 
>> merge or a rebase. The pull request now contains 506 commits:
>> 
>>  - merge latest changes from master branch
>>  - http3: update H3InsertionsLimitTest to start after receival of client 
>> settings
>>  - merge latest changes from master branch
>>  - quic: separate out the idle termination timer and the STREAM_DATA_BLOCKED 
>> timer
>>  - quic: simplify idle timeout management
>>  - http3: rely on the sole isOpen() method instead of isOpen() and isClosed()
>>  - quic: do not let h3 idle (in pool) timeout to influence the quic 
>> transport idle timeout
>>  - merge latest changes from master branch
>>  - http3: improve H3ConnectionPoolTest.java
>>  - Fix snippet
>>  - ... and 496 more: https://git.openjdk.org/jdk/compare/8d33ea73...d4984d5e
>
> src/java.net.http/share/classes/jdk/internal/net/http/hpack/ISO_8859_1.java 
> line 43:
> 
>> 41: // The encoding is simple and well known: 1 byte <-> 1 char
>> 42: //
>> 43: final class ISO_8859_1 {
> 
> OK - this class is actually used in QPack as well as HPack. Same for the 
> `QuickHuffman` class below (notice that this is Quick - not Quic ;-) )
> 
> It might be interesting to investigate if we could pull out these classes to 
> a common location to break the tie between QPack and HPack. Something to 
> think about after integration maybe.

@AlekseiEfimov I am resolving this conversation now. Something we could think 
about later!

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

PR Review Comment: https://git.openjdk.org/jdk/pull/24751#discussion_r2166201911

Reply via email to