mktree: drop unused length parameter

The mktree_line() function does not actually look at the "len" parameter it is passed, and assumes the buffer it receives is NUL-terminated. Since the caller always passes a strbuf, this will be true. Let's drop the useless parameter. Signed-off-by: Jeff King <peff@peff.net> Signed-off-by: Junio C Hamano <gitster@pobox.com>

Jeff King committed May 9, 2019 at 17:30 UTC be27fb7b287a81b5b3c889121eab99d344272e8b
1 file changed +2 -2
builtin/mktree.c
+2 -2
@@ -67,7 +67,7 @@ static const char *mktree_usage[] = {
67 NULL
68 };
69
70 -static void mktree_line(char *buf, size_t len, int nul_term_line, int allow_missing)
70 +static void mktree_line(char *buf, int nul_term_line, int allow_missing)
71 {
72 char *ptr, *ntr;
73 const char *p;
@@ -172,7 +172,7 @@ int cmd_mktree(int ac, const char **av, const char *prefix)
172 break;
173 die("input format error: (blank line only valid in batch mode)");
174 }
175 - mktree_line(sb.buf, sb.len, nul_term_line, allow_missing);
175 + mktree_line(sb.buf, nul_term_line, allow_missing);
176 }
177 if (is_batch_mode && got_eof && used < 1) {
178 /*