[PATCH] revision walker --boundary: show boundaries again
- From
Johannes Schindelin <johannes.schindelin@gmx.de>
- Date
- Mar 6, 2007, 11:54 UTC
- Message-ID
- <Pine.LNX.4.63.0703061252110.22628@wbgn013.biozentrum.uni-wuerzburg.de>
Without this patch, "git rev-list --boundary master^..master" shows only master's object name, and nothing else.
It is mainly an indentation patch, because it delays exit in get_revision() until after revs->boundary was checked.
Signed-off-by: Johannes Schindelin <Johannes.Schindelin@gmx.de> ---
Sorry, I don't have time to write a test case, as I have to run.
revision.c | 46 +++++++++++++++++++++++----------------------- 1 files changed, 23 insertions(+), 23 deletions(-)
diff --git a/revision.c b/revision.c index 35a1711..b6b9fc0 100644 --- a/revision.c +++ b/revision.c @@ -1301,31 +1301,31 @@ struct commit *get_revision(struct rev_info *revs) /* * Now pick up what they want to give us */ - if (!(c = get_revision_1(revs))) - return NULL; - while (0 < revs->skip_count) { - revs->skip_count--; - c = get_revision_1(revs); - if (!c) - break; - } + if ((c = get_revision_1(revs))) { + while (0 < revs->skip_count) { + revs->skip_count--; + c = get_revision_1(revs); + if (!c) + break; + } - /* - * Check the max_count. - */ - switch (revs->max_count) { - case -1: - break; - case 0: - /* Although we grabbed it, it is not shown. */ - c = NULL; - break; - default: - revs->max_count--; - } + /* + * Check the max_count. + */ + switch (revs->max_count) { + case -1: + break; + case 0: + /* Although we grabbed it, it is not shown. */ + c = NULL; + break; + default: + revs->max_count--; + } - if (c) - c->object.flags |= SHOWN; + if (c) + c->object.flags |= SHOWN; + } if (!revs->boundary) { return c;
-- 1.5.0.3.2551.gf95a-dirty