Hello there,

code snippet in the definition of ngx_chain_add_copy in ngx_buf.c:


ll = chain;

for (cl = *chain; cl; cl = cl->next) {
ll = &cl->next;
}


Why is ll assigned repeatedly? I'm sorry for failed thinking out any necessity.

And I modified the above as the following. Is it OK?


if (*chain) {
for (cl = *chain; cl->next; cl = cl->next) { /* void */ }
ll = &cl->next;
} else {
ll = chain;
}


Thank you very much.