[PATCH 10/11] HTTP: removed unused nxt_http_cookie_t structure.

Alejandro Colomar alx.manpages at gmail.com
Thu Jun 16 16:03:38 UTC 2022


Hi Andrew,

On 6/16/22 03:01, Andrew Clayton wrote:
> Turns out that struct nxt_http_cookie_t is completely unused.

I think this was added to give support to a future feature that will be 
added soon, but I don't remember the details exactly.

Anyway, this patch is good on its own, so if the code continues not 
using this type in the future, I'm fine with applying this.  We'll see 
soon, I guess.

Cheers,

Alex

> ---
>   src/nxt_http_route.c | 9 ---------
>   1 file changed, 9 deletions(-)
> 
> diff --git a/src/nxt_http_route.c b/src/nxt_http_route.c
> index f702399..d1fe298 100644
> --- a/src/nxt_http_route.c
> +++ b/src/nxt_http_route.c
> @@ -79,15 +79,6 @@ typedef struct {
>   } nxt_http_route_pattern_t;
>   
>   
> -typedef struct {
> -    uint16_t                       hash;
> -    uint16_t                       name_length;
> -    uint32_t                       value_length;
> -    u_char                         *name;
> -    u_char                         *value;
> -} nxt_http_cookie_t;
> -
> -
>   struct nxt_http_route_rule_s {
>       /* The object must be the first field. */
>       nxt_http_route_object_t        object:8;

-- 
Alejandro Colomar
<http://www.alejandro-colomar.es/>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: OpenPGP_signature
Type: application/pgp-signature
Size: 833 bytes
Desc: OpenPGP digital signature
URL: <http://mailman.nginx.org/pipermail/unit/attachments/20220616/ddf8466e/attachment.bin>


More information about the unit mailing list