submodule: use diff_tree_combined_merge() instead of diff_tree_combined()
authorRené Scharfe <rene.scharfe@lsrfire.ath.cx>
Sat, 17 Dec 2011 10:27:19 +0000 (11:27 +0100)
committerJunio C Hamano <gitster@pobox.com>
Sun, 18 Dec 2011 02:23:24 +0000 (18:23 -0800)
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>
submodule.c

index 788d5327ac09f967e874626c7a077fec7eb29e11..9a2806067954c55a27c068706b5bfe67a1189fd5 100644 (file)
@@ -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);