diff options
author | Junio C Hamano <gitster@pobox.com> | 2014-01-10 10:33:45 -0800 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2014-01-10 10:33:45 -0800 |
commit | 4243b2d1e45a8d0d9b1ccc9d860235e7b9543a7f (patch) | |
tree | 30a290c6a782b9717fcf5cf192800693094a6579 | |
parent | Merge branch 'ta/format-user-manual-as-an-article' (diff) | |
parent | get_octopus_merge_bases(): cleanup redundant variable (diff) | |
download | tgif-4243b2d1e45a8d0d9b1ccc9d860235e7b9543a7f.tar.xz |
Merge branch 'vm/octopus-merge-bases-simplify'
* vm/octopus-merge-bases-simplify:
get_octopus_merge_bases(): cleanup redundant variable
-rw-r--r-- | commit.c | 36 |
1 files changed, 18 insertions, 18 deletions
@@ -841,26 +841,26 @@ static struct commit_list *merge_bases_many(struct commit *one, int n, struct co struct commit_list *get_octopus_merge_bases(struct commit_list *in) { struct commit_list *i, *j, *k, *ret = NULL; - struct commit_list **pptr = &ret; - for (i = in; i; i = i->next) { - if (!ret) - pptr = &commit_list_insert(i->item, pptr)->next; - else { - struct commit_list *new = NULL, *end = NULL; - - for (j = ret; j; j = j->next) { - struct commit_list *bases; - bases = get_merge_bases(i->item, j->item, 1); - if (!new) - new = bases; - else - end->next = bases; - for (k = bases; k; k = k->next) - end = k; - } - ret = new; + if (!in) + return ret; + + commit_list_insert(in->item, &ret); + + for (i = in->next; i; i = i->next) { + struct commit_list *new = NULL, *end = NULL; + + for (j = ret; j; j = j->next) { + struct commit_list *bases; + bases = get_merge_bases(i->item, j->item, 1); + if (!new) + new = bases; + else + end->next = bases; + for (k = bases; k; k = k->next) + end = k; } + ret = new; } return ret; } |