From: Vasily Makarov Date: Fri, 3 Jan 2014 14:45:46 +0000 (+0400) Subject: get_octopus_merge_bases(): cleanup redundant variable X-Git-Tag: v1.9-rc0~19^2 X-Git-Url: https://git.lorimer.id.au/gitweb.git/diff_plain/6bc76725eaa2f4bb8c5fad47f843425d9160e7f3?ds=inline;hp=--cc get_octopus_merge_bases(): cleanup redundant variable pptr is needless. Some related code got cleaned as well. Signed-off-by: Vasily Makarov Signed-off-by: Junio C Hamano --- 6bc76725eaa2f4bb8c5fad47f843425d9160e7f3 diff --git a/commit.c b/commit.c index 5df1df7351..249a83b0cc 100644 --- a/commit.c +++ b/commit.c @@ -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; }