Skip to content

Commit

Permalink
[PATCH] fetch.c: Remove redundant TO_FETCH flag
Browse files Browse the repository at this point in the history
The TO_FETCH flag also became redundant after adding the SEEN flag -
it was set and checked in process() to prevent adding the same object
to process_queue multiple times, but now SEEN guards against this.

Signed-off-by: Sergey Vlasov <[email protected]>
Signed-off-by: Junio C Hamano <[email protected]>
  • Loading branch information
sigprof authored and Junio C Hamano committed Sep 21, 2005
1 parent 125beb1 commit c5c0f45
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions fetch.c
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,6 @@ static int process_tree(struct tree *tree)
}

#define COMPLETE 1U
#define TO_FETCH 2U
#define TO_SCAN 4U
#define SEEN 16U

Expand Down Expand Up @@ -144,11 +143,10 @@ static int process(struct object *obj)
obj->flags |= TO_SCAN;
return 0;
}
if (obj->flags & (COMPLETE | TO_FETCH))
if (obj->flags & COMPLETE)
return 0;
object_list_insert(obj, process_queue_end);
process_queue_end = &(*process_queue_end)->next;
obj->flags |= TO_FETCH;

prefetch(obj->sha1);

Expand Down

0 comments on commit c5c0f45

Please sign in to comment.