summaryrefslogtreecommitdiff
path: root/submodule.c
diff options
context:
space:
mode:
authorRené Scharfe <rene.scharfe@lsrfire.ath.cx>2011-12-17 11:27:19 +0100
committerJunio C Hamano <gitster@pobox.com>2011-12-17 18:23:24 -0800
commit78e98eaf5ee49bc6d62ef9bdf87066528bfba248 (patch)
treefee05f1eab7498d0bac3271986185bd80fc2b8d4 /submodule.c
parent82889295e76200714d76e0f7beb07b6b9e3b5a88 (diff)
downloadgit-78e98eaf5ee49bc6d62ef9bdf87066528bfba248.tar.gz
submodule: use diff_tree_combined_merge() instead of diff_tree_combined()rs/diff-tree-combined-clean-up
Use diff_tree_combined_merge() instead of open-coding it. Signed-off-by: René Scharfe <rene.scharfe@lsrfire.ath.cx> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'submodule.c')
-rw-r--r--submodule.c14
1 files changed, 3 insertions, 11 deletions
diff --git a/submodule.c b/submodule.c
index 788d5327ac..9a28060679 100644
--- a/submodule.c
+++ b/submodule.c
@@ -371,23 +371,15 @@ static void collect_submodules_from_diff(struct diff_queue_struct *q,
}
-static void commit_need_pushing(struct commit *commit, struct commit_list *parent, int *needs_pushing)
+static void commit_need_pushing(struct commit *commit, int *needs_pushing)
{
- struct sha1_array parents = SHA1_ARRAY_INIT;
struct rev_info rev;
- while (parent) {
- sha1_array_append(&parents, parent->item->object.sha1);
- parent = parent->next;
- }
-
init_revisions(&rev, NULL);
rev.diffopt.output_format |= DIFF_FORMAT_CALLBACK;
rev.diffopt.format_callback = collect_submodules_from_diff;
rev.diffopt.format_callback_data = needs_pushing;
- diff_tree_combined(commit->object.sha1, &parents, 1, &rev);
-
- sha1_array_clear(&parents);
+ diff_tree_combined_merge(commit, 1, &rev);
}
int check_submodule_needs_pushing(unsigned char new_sha1[20], const char *remotes_name)
@@ -410,7 +402,7 @@ int check_submodule_needs_pushing(unsigned char new_sha1[20], const char *remote
die("revision walk setup failed");
while ((commit = get_revision(&rev)) && !needs_pushing)
- commit_need_pushing(commit, commit->parents, &needs_pushing);
+ commit_need_pushing(commit, &needs_pushing);
free(sha1_copy);
strbuf_release(&remotes_arg);