Subject: Re: [PATCH v3] sideband.c: remove redundant 'NEEDSWORK' tag

From: Junio C Hamano <[email-masked]>
Date: Wed, 27 Dec 2023 14:22:51 -0800

"Chandra Pratap via GitGitGadget" <[email-masked]> writes:

>       - * NEEDSWORK: use "size_t n" instead for clarity.
>      ++ * It is fine to use "int n" here instead of "size_t n" as all calls to this
>      ++ * function pass an 'int' parameter.

This does not sound like a sufficient justification, though.

We should also explain why "int" is good enough for these callers.
Otherwise, using size_t throughout the callchain would become
another viable solution.