Question about ngx_chain_writer
eran.kornblau at kaltura.com
Thu Jun 6 13:06:38 UTC 2019
Looking at ngx_chain_writer, there's something that looks a bit off to me, this line seems redundant -
The first loop in the function adds the input chain (in) to the end of ctx->out.
The second loop in the function calculates the total size of ctx->out, so it will add the size of the 'in' buffers again.
I verified in gdb that size is indeed x2 the size of the chain.
This isn't causing any problem since size is only compared to zero, but seems a bit confusing...
It may have been better to leave the 'size += ngx_buf_size(in->buf);' but reverse the order of the 2 loops,
so that the input buffers will be validated only once.
Does this make sense or did I get something wrong?
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the nginx-devel