[PATCH 2 of 3] QUIC: simplified stream initialization

Sergey Kandaurov pluknet at nginx.com
Fri Dec 10 14:00:22 UTC 2021


> On 25 Nov 2021, at 17:20, Roman Arutyunyan <arut at nginx.com> wrote:
> 
> # HG changeset patch
> # User Roman Arutyunyan <arut at nginx.com>
> # Date 1637693300 -10800
> #      Tue Nov 23 21:48:20 2021 +0300
> # Branch quic
> # Node ID 3d2354bfa1a2a257b9f73772ad0836585be85a6c
> # Parent  5b03ffd757804542daec73188a509b02e6b2c596
> QUIC: simplified stream initialization.
> 
> After creation, a client stream is added to qc->streams.uninitialized queue.
> After initialization it's removed from the queue.  If a stream is never
> initialized, it is freed in ngx_quic_close_streams().  Stream initializer
> is now set as read event handler in stream connection.
> 
> Previously qc->streams.uninitialized was used for delayed stream

was used *solely*? with the patch, it's still used to

> initialization.
> 
> The change makes is possible not to handle separately the case of a new stream

s/is/it/

> in stream-related frame handlers.  It makes these handlers simpler since new
> streams and existing streams are now handled by the same code.
> 

The change looks good.

-- 
Sergey Kandaurov



More information about the nginx-devel mailing list