aboutsummaryrefslogtreecommitdiffstats
path: root/builtin
diff options
context:
space:
mode:
authorÆvar Arnfjörð Bjarmason <avarab@gmail.com>2022-04-13 22:01:50 +0200
committerJunio C Hamano <gitster@pobox.com>2022-04-13 23:56:09 -0700
commit689a8e80dd4f7b3a20be88146bdea833c7759603 (patch)
tree3b59de124fe5c41ba3e9e78415b60c66d1b0a61c /builtin
parentrevisions API: have release_revisions() release "grep_filter" (diff)
downloadgit-689a8e80dd4f7b3a20be88146bdea833c7759603.tar.gz
git-689a8e80dd4f7b3a20be88146bdea833c7759603.zip
revisions API: have release_revisions() release "prune_data"
Extend the the release_revisions() function so that it frees the "prune_data" in the "struct rev_info". This means that any code that calls "release_revisions()" already can get rid of adjacent calls to clear_pathspec(). Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin')
-rw-r--r--builtin/add.c1
-rw-r--r--builtin/stash.c2
2 files changed, 0 insertions, 3 deletions
diff --git a/builtin/add.c b/builtin/add.c
index 115a26ea63..fc729e14c1 100644
--- a/builtin/add.c
+++ b/builtin/add.c
@@ -142,7 +142,6 @@ int add_files_to_cache(const char *prefix,
rev.diffopt.flags.override_submodule_config = 1;
rev.max_count = 0; /* do not compare unmerged paths with stage #2 */
run_diff_files(&rev, DIFF_RACY_IS_MODIFIED);
- clear_pathspec(&rev.prune_data);
release_revisions(&rev);
return !!data.add_errors;
}
diff --git a/builtin/stash.c b/builtin/stash.c
index 7c9c5751f5..e6b4c12ebb 100644
--- a/builtin/stash.c
+++ b/builtin/stash.c
@@ -1064,7 +1064,6 @@ static int check_changes_tracked_files(const struct pathspec *ps)
}
done:
- clear_pathspec(&rev.prune_data);
release_revisions(&rev);
return ret;
}
@@ -1276,7 +1275,6 @@ static int stash_working_tree(struct stash_info *info, const struct pathspec *ps
done:
discard_index(&istate);
- clear_pathspec(&rev.prune_data);
release_revisions(&rev);
strbuf_release(&diff_output);
remove_path(stash_index_path.buf);