From: Fredrik Gustafsson Date: Wed, 15 Jun 2011 18:08:38 GMT Subject: [PATCH] Use correct value when hinting strbuf_read() Message-ID: <1308161318-25637-1-git-send-email-iveqy@iveqy.com> The git strbuf allows for each read to hint about the size of the string. In this case the the string can never be longer than 41 characters, as it cannot contain more than a single hex-sha1 and a newline. So let's use 41 instead of 1024 to reduce the memory footprint. Signed-off-by: Fredrik Gustafsson Mentored-by: Jens Lehmann Mentored-by: Heiko Voigt --- submodule.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/submodule.c b/submodule.c index b6dec70..86baf42 100644 --- a/submodule.c +++ b/submodule.c @@ -326,7 +326,7 @@ static int is_submodule_commit_present(const char *path, unsigned char sha1[20]) cp.no_stdin = 1; cp.out = -1; cp.dir = path; - if (!run_command(&cp) && !strbuf_read(&buf, cp.out, 1024)) + if (!run_command(&cp) && !strbuf_read(&buf, cp.out, 41)) is_present = 1; close(cp.out); -- 1.7.5.1.229.g455f