diff options
| author | Chandra Pratap <chandrapratap3519@gmail.com> | 2023-12-28 11:01:00 +0300 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2023-12-28 18:59:12 +0300 |
| commit | 03bcc93769bd5e3b14094c36e3d466587f922274 (patch) | |
| tree | d53628c85ca160288fceed9b8bcea53cd96ba8df /sideband.c | |
| parent | 0d1bd1dfb37ef25e1911777c94129fc769ffec38 (diff) | |
sideband.c: remove redundant 'NEEDSWORK' tag
Signed-off-by: Chandra Pratap <chandrapratap3519@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'sideband.c')
| -rw-r--r-- | sideband.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/sideband.c b/sideband.c index 85bddfdcd4..cdbe0789f0 100644 --- a/sideband.c +++ b/sideband.c @@ -66,7 +66,10 @@ void list_config_color_sideband_slots(struct string_list *list, const char *pref * of the line. This should be called for a single line only, which is * passed as the first N characters of the SRC array. * - * 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. Additionally, the buffer involved in + * storing these 'int' values takes input from a packet via the pkt-line + * interface, which is capable of transferring only 64kB at a time. */ static void maybe_colorize_sideband(struct strbuf *dest, const char *src, int n) { |