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@xxxxxx>
---
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
-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html