diff options
author | Junio C Hamano <gitster@pobox.com> | 2016-10-18 12:42:55 -0700 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2016-10-18 12:43:13 -0700 |
commit | de2968947a9b7c0f933856f0be84f7db84d190b6 (patch) | |
tree | 584d99085c1b9ffe468a447e58c1707ba3bcbeb4 | |
parent | 4397cafc7895c857cf38a920125a91f1cb13060c (diff) | |
download | git-de2968947a9b7c0f933856f0be84f7db84d190b6.tar.gz |
sha1_name: remove ONELINE_SEEN bit
28a4d94044 ("object name: introduce ':/<oneline prefix>' notation",
2007-02-24) started using its own bit from object->flags to mark
commits used while parsing the ":/token" extended SHA-1 syntax to
name a commit temporarily, and this was kept even when f7bff00314
("sha1_name.c: fix parsing of ":/token" syntax", 2010-08-02) found
and fixed a bug in its implementation.
The use of that flag bit, however, is limited to a single function,
get_sha1_oneline(), which first sets it for the commits sitting at
the tips of refs, uses the bit to avoid duplicate traversal while
walking the history, and then cleans the bit from all commits it
walked.
Which is exactly what the general-purpose TMP_MARK bit meant to be
used for isolated case was invented for. Replace ONELINE_SEEN with
TMP_MARK and retire the former.
Signed-off-by: Junio C Hamano <gitster@pobox.com>
-rw-r--r-- | object.h | 1 | ||||
-rw-r--r-- | sha1_name.c | 10 |
2 files changed, 4 insertions, 7 deletions
@@ -37,7 +37,6 @@ struct object_array { * bundle.c: 16 * http-push.c: 16-----19 * commit.c: 16-----19 - * sha1_name.c: 20 */ #define FLAG_BITS 27 diff --git a/sha1_name.c b/sha1_name.c index ca7ddd6f2c..fa0e6701a3 100644 --- a/sha1_name.c +++ b/sha1_name.c @@ -7,6 +7,7 @@ #include "refs.h" #include "remote.h" #include "dir.h" +#include "revision.h" static int get_sha1_oneline(const char *, unsigned char *, struct commit_list *); @@ -855,9 +856,6 @@ static int get_sha1_1(const char *name, int len, unsigned char *sha1, unsigned l * For future extension, all other sequences beginning with ':/!' are reserved. */ -/* Remember to update object flag allocation in object.h */ -#define ONELINE_SEEN (1u<<20) - static int handle_one_ref(const char *path, const struct object_id *oid, int flag, void *cb_data) { @@ -899,7 +897,7 @@ static int get_sha1_oneline(const char *prefix, unsigned char *sha1, return -1; for (l = list; l; l = l->next) { - l->item->object.flags |= ONELINE_SEEN; + l->item->object.flags |= TMP_MARK; commit_list_insert(l->item, &backup); } while (list) { @@ -907,7 +905,7 @@ static int get_sha1_oneline(const char *prefix, unsigned char *sha1, struct commit *commit; int matches; - commit = pop_most_recent_commit(&list, ONELINE_SEEN); + commit = pop_most_recent_commit(&list, TMP_MARK); if (!parse_object(commit->object.oid.hash)) continue; buf = get_commit_buffer(commit, NULL); @@ -924,7 +922,7 @@ static int get_sha1_oneline(const char *prefix, unsigned char *sha1, regfree(®ex); free_commit_list(list); for (l = backup; l; l = l->next) - clear_commit_marks(l->item, ONELINE_SEEN); + clear_commit_marks(l->item, TMP_MARK); free_commit_list(backup); return found ? 0 : -1; } |