summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorSergey Vlasov <vsu@altlinux.ru>2005-09-21 16:33:54 (GMT)
committerJunio C Hamano <junkio@cox.net>2005-09-23 04:52:10 (GMT)
commit80077f071614ef1775472eb54d59c071e15d5784 (patch)
treefe801ad6c55995e978cdebf80ba88b6ba42c6b4a
parentbc7ccfd2d7342bf81d344c4215a17ebbb64d9212 (diff)
downloadgit-80077f071614ef1775472eb54d59c071e15d5784.zip
git-80077f071614ef1775472eb54d59c071e15d5784.tar.gz
git-80077f071614ef1775472eb54d59c071e15d5784.tar.bz2
[PATCH] fetch.c: Remove useless lookup_object_type() call in process()
In all places where process() is called except the one in pull() (which is executed only once) the pointer to the object is already available, so pass it as the argument to process() instead of sha1 and avoid an unneeded call to lookup_object_type(). Signed-off-by: Sergey Vlasov <vsu@altlinux.ru> Signed-off-by: Junio C Hamano <junkio@cox.net>
-rw-r--r--fetch.c23
1 files changed, 10 insertions, 13 deletions
diff --git a/fetch.c b/fetch.c
index af9a013..be3dc0a 100644
--- a/fetch.c
+++ b/fetch.c
@@ -33,7 +33,7 @@ static void report_missing(const char *what, const unsigned char *missing)
what, missing_hex, sha1_to_hex(current_commit_sha1));
}
-static int process(unsigned char *sha1, const char *type);
+static int process(struct object *obj);
static int process_tree(struct tree *tree)
{
@@ -46,8 +46,7 @@ static int process_tree(struct tree *tree)
tree->entries = NULL;
while (entry) {
struct tree_entry_list *next = entry->next;
- if (process(entry->item.any->sha1,
- entry->directory ? tree_type : blob_type))
+ if (process(entry->item.any))
return -1;
free(entry);
entry = next;
@@ -79,7 +78,7 @@ static int process_commit(struct commit *commit)
pull_say("walk %s\n", sha1_to_hex(commit->object.sha1));
if (get_tree) {
- if (process(commit->tree->object.sha1, tree_type))
+ if (process(&commit->tree->object))
return -1;
if (!get_all)
get_tree = 0;
@@ -87,7 +86,7 @@ static int process_commit(struct commit *commit)
if (get_history) {
struct commit_list *parents = commit->parents;
for (; parents; parents = parents->next) {
- if (process(parents->item->object.sha1, commit_type))
+ if (process(&parents->item->object))
return -1;
}
}
@@ -98,7 +97,7 @@ static int process_tag(struct tag *tag)
{
if (parse_tag(tag))
return -1;
- return process(tag->tagged->sha1, NULL);
+ return process(tag->tagged);
}
static struct object_list *process_queue = NULL;
@@ -133,12 +132,10 @@ static int process_object(struct object *obj)
obj->type, sha1_to_hex(obj->sha1));
}
-static int process(unsigned char *sha1, const char *type)
+static int process(struct object *obj)
{
- struct object *obj = lookup_object_type(sha1, type);
-
- if (has_sha1_file(sha1)) {
- parse_object(sha1);
+ if (has_sha1_file(obj->sha1)) {
+ parse_object(obj->sha1);
/* We already have it, so we should scan it now. */
if (obj->flags & (SCANNED | TO_SCAN))
return 0;
@@ -153,7 +150,7 @@ static int process(unsigned char *sha1, const char *type)
process_queue_end = &(*process_queue_end)->next;
obj->flags |= TO_FETCH;
- prefetch(sha1);
+ prefetch(obj->sha1);
return 0;
}
@@ -228,7 +225,7 @@ int pull(char *target)
if (interpret_target(target, sha1))
return error("Could not interpret %s as something to pull",
target);
- if (process(sha1, NULL))
+ if (process(lookup_unknown_object(sha1)))
return -1;
if (loop())
return -1;