diff options
author | Jeff King <peff@peff.net> | 2015-08-10 05:37:55 -0400 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2015-08-10 15:37:14 -0700 |
commit | 0ea68e429647cc5400fe8fa257056083b0a6459d (patch) | |
tree | e7bf8ddf1a1f6400f74ef3f3a82fae58434f3cd1 /t/t5515/refs.br-branches-one-merge | |
parent | find_hook: keep our own static buffer (diff) | |
download | tgif-0ea68e429647cc5400fe8fa257056083b0a6459d.tar.xz |
get_repo_path: refactor path-allocation
The get_repo_path function calls mkpath() and then does some
non-trivial operations on it, like calling
is_git_directory() and read_gitfile(). These are actually
OK (they do not use more pathname static buffers
themselves), but it takes a fair bit of work to verify.
Let's use our own strbuf to store the path, and we can
simply reuse it for each iteration of the loop (we can even
avoid rewriting the beginning part, since we are trying a
series of suffixes).
To make the strbuf cleanup easier, we split out a thin
wrapper. As a bonus, this wrapper can factor out the
canonicalization that happens in all of the early-return
code paths.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't/t5515/refs.br-branches-one-merge')
0 files changed, 0 insertions, 0 deletions