summaryrefslogtreecommitdiff
path: root/builtin/mktree.c
diff options
context:
space:
mode:
authorLibravatar Jeff King <peff@peff.net>2019-05-09 17:30:37 -0400
committerLibravatar Junio C Hamano <gitster@pobox.com>2019-05-13 14:22:54 +0900
commitbe27fb7b287a81b5b3c889121eab99d344272e8b (patch)
tree37e4e5b88952135bc35983f0bf02b2f123d90e59 /builtin/mktree.c
parentwt-status: drop unused status parameter (diff)
downloadtgif-be27fb7b287a81b5b3c889121eab99d344272e8b.tar.xz
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>
Diffstat (limited to 'builtin/mktree.c')
-rw-r--r--builtin/mktree.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/builtin/mktree.c b/builtin/mktree.c
index 94e82b8504..891991b00d 100644
--- a/builtin/mktree.c
+++ b/builtin/mktree.c
@@ -67,7 +67,7 @@ static const char *mktree_usage[] = {
NULL
};
-static void mktree_line(char *buf, size_t len, int nul_term_line, int allow_missing)
+static void mktree_line(char *buf, int nul_term_line, int allow_missing)
{
char *ptr, *ntr;
const char *p;
@@ -172,7 +172,7 @@ int cmd_mktree(int ac, const char **av, const char *prefix)
break;
die("input format error: (blank line only valid in batch mode)");
}
- mktree_line(sb.buf, sb.len, nul_term_line, allow_missing);
+ mktree_line(sb.buf, nul_term_line, allow_missing);
}
if (is_batch_mode && got_eof && used < 1) {
/*