diff options
author | Junio C Hamano <gitster@pobox.com> | 2016-06-02 14:09:22 -0700 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2016-06-02 14:09:22 -0700 |
commit | ed6e8038f98e7f15e571a9a291ee4acf438b4dc5 (patch) | |
tree | 82560db1d8fef5db3c449213f992bd26a5473ac5 /builtin/blame.c | |
parent | 60bd4b1c513bb652cdffad44382046ca872140eb (diff) | |
download | git-ed6e8038f98e7f15e571a9a291ee4acf438b4dc5.tar.gz git-ed6e8038f98e7f15e571a9a291ee4acf438b4dc5.tar.xz |
pathspec: rename free_pathspec() to clear_pathspec()
The function takes a pointer to a pathspec structure, and releases
the resources held by it, but does not free() the structure itself.
Such a function should be called "clear", not "free".
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin/blame.c')
-rw-r--r-- | builtin/blame.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/builtin/blame.c b/builtin/blame.c index 21f42b0b6..759d84aff 100644 --- a/builtin/blame.c +++ b/builtin/blame.c @@ -609,7 +609,7 @@ static struct origin *find_origin(struct scoreboard *sb, } } diff_flush(&diff_opts); - free_pathspec(&diff_opts.pathspec); + clear_pathspec(&diff_opts.pathspec); return porigin; } @@ -651,7 +651,7 @@ static struct origin *find_rename(struct scoreboard *sb, } } diff_flush(&diff_opts); - free_pathspec(&diff_opts.pathspec); + clear_pathspec(&diff_opts.pathspec); return porigin; } @@ -1343,7 +1343,7 @@ static void find_copy_in_parent(struct scoreboard *sb, } while (unblamed); target->suspects = reverse_blame(leftover, NULL); diff_flush(&diff_opts); - free_pathspec(&diff_opts.pathspec); + clear_pathspec(&diff_opts.pathspec); } /* |