aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJohannes Schindelin <Johannes.Schindelin@gmx.de>2006-06-27 22:38:04 +0200
committerJunio C Hamano <junkio@cox.net>2006-06-27 16:03:59 -0700
commit81db094107df9a73a0be4538a74a26115a84dd13 (patch)
treeb0f323fa4f2fb82be914643867dc86f2066a02ff
parent8780bd8fd2fb9a85c39de8d7d3105b94c6a9cf40 (diff)
downloadgit-81db094107df9a73a0be4538a74a26115a84dd13.tar.gz
git-81db094107df9a73a0be4538a74a26115a84dd13.tar.xz
format-patch: use clear_commit_marks() instead of some ad-hockery
It is cleaner, and it describes better what the idea behind the code is. Signed-off-by: Johannes Schindelin <Johannes.Schindelin@gmx.de> Signed-off-by: Junio C Hamano <junkio@cox.net>
-rw-r--r--builtin-log.c14
1 files changed, 4 insertions, 10 deletions
diff --git a/builtin-log.c b/builtin-log.c
index 4ee5891df..f9515a8a4 100644
--- a/builtin-log.c
+++ b/builtin-log.c
@@ -160,15 +160,6 @@ static void reopen_stdout(struct commit *commit, int nr, int keep_subject)
freopen(filename, "w", stdout);
}
-static void reset_all_objects_flags()
-{
- int i;
-
- for (i = 0; i < obj_allocs; i++)
- if (objs[i])
- objs[i]->flags = 0;
-}
-
static int get_patch_id(struct commit *commit, struct diff_options *options,
unsigned char *sha1)
{
@@ -220,7 +211,10 @@ static void get_patch_ids(struct rev_info *rev, struct diff_options *options)
}
/* reset for next revision walk */
- reset_all_objects_flags();
+ clear_commit_marks((struct commit *)o1,
+ SEEN | UNINTERESTING | SHOWN | ADDED);
+ clear_commit_marks((struct commit *)o2,
+ SEEN | UNINTERESTING | SHOWN | ADDED);
o1->flags = flags1;
o2->flags = flags2;
}