undef/git-commit-vandalism
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Move old index entry removal from "unpack_trees()" into the individual functions

Browse Source 
This makes no changes to current code, but it allows the individual merge
functions to decide what to do about the old entry.  They might decide to
update it in place, rather than force them to always delete and re-add it.

Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
This commit is contained in:
Linus Torvalds 2007-08-10 12:15:54 -07:00 committed by Junio C Hamano
parent 79f5e0645a
commit b48d5a050a
2 changed files with 29 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

29
unpack-trees.c
View File

@ -58,10 +58,17 @@ static int entcmp(const char *name1, int dir1, const char *name2, int dir2)
return ret; return ret;
} }
static inline void remove_entry(int remove)
{
if (remove >= 0)
remove_cache_entry_at(remove);
}
static int unpack_trees_rec(struct tree_entry_list **posns, int len, static int unpack_trees_rec(struct tree_entry_list **posns, int len,
const char *base, struct unpack_trees_options *o, const char *base, struct unpack_trees_options *o,
struct tree_entry_list *df_conflict_list) struct tree_entry_list *df_conflict_list)
{ {
int remove;
int baselen = strlen(base); int baselen = strlen(base);
int src_size = len + 1; int src_size = len + 1;
int i_stk = i_stk; int i_stk = i_stk;
@ -145,10 +152,11 @@ static int unpack_trees_rec(struct tree_entry_list **posns, int len,
subposns = xcalloc(len, sizeof(struct tree_list_entry *)); subposns = xcalloc(len, sizeof(struct tree_list_entry *));
remove = -1;
if (cache_name && !strcmp(cache_name, first)) { if (cache_name && !strcmp(cache_name, first)) {
any_files = 1; any_files = 1;
src[0] = active_cache[o->pos]; src[0] = active_cache[o->pos];
remove_cache_entry_at(o->pos); remove = o->pos;
} }
for (i = 0; i < len; i++) { for (i = 0; i < len; i++) {
@ -214,13 +222,14 @@ static int unpack_trees_rec(struct tree_entry_list **posns, int len,
printf("\n"); printf("\n");
} }
#endif #endif
ret = o->fn(src, o); ret = o->fn(src, o, remove);
#if DBRT_DEBUG > 1 #if DBRT_DEBUG > 1
printf("Added %d entries\n", ret); printf("Added %d entries\n", ret);
#endif #endif
o->pos += ret; o->pos += ret;
} else { } else {
remove_entry(remove);
for (i = 0; i < src_size; i++) { for (i = 0; i < src_size; i++) {
if (src[i]) { if (src[i]) {
add_cache_entry(src[i], ADD_CACHE_OK_TO_ADD|ADD_CACHE_SKIP_DFCHECK); add_cache_entry(src[i], ADD_CACHE_OK_TO_ADD|ADD_CACHE_SKIP_DFCHECK);
@ -641,7 +650,8 @@ static void show_stage_entry(FILE *o,
#endif #endif
int threeway_merge(struct cache_entry **stages, int threeway_merge(struct cache_entry **stages,
struct unpack_trees_options *o) struct unpack_trees_options *o,
int remove)
{ {
struct cache_entry *index; struct cache_entry *index;
struct cache_entry *head; struct cache_entry *head;
@ -657,6 +667,7 @@ int threeway_merge(struct cache_entry **stages,
int no_anc_exists = 1; int no_anc_exists = 1;
int i; int i;
remove_entry(remove);
for (i = 1; i < o->head_idx; i++) { for (i = 1; i < o->head_idx; i++) {
if (!stages[i] || stages[i] == o->df_conflict_entry) if (!stages[i] || stages[i] == o->df_conflict_entry)
any_anc_missing = 1; any_anc_missing = 1;
@ -809,12 +820,14 @@ int threeway_merge(struct cache_entry **stages,
* *
*/ */
int twoway_merge(struct cache_entry **src, int twoway_merge(struct cache_entry **src,
struct unpack_trees_options *o) struct unpack_trees_options *o,
int remove)
{ {
struct cache_entry *current = src[0]; struct cache_entry *current = src[0];
struct cache_entry *oldtree = src[1]; struct cache_entry *oldtree = src[1];
struct cache_entry *newtree = src[2]; struct cache_entry *newtree = src[2];
remove_entry(remove);
if (o->merge_size != 2) if (o->merge_size != 2)
return error("Cannot do a twoway merge of %d trees", return error("Cannot do a twoway merge of %d trees",
o->merge_size); o->merge_size);
@ -868,11 +881,13 @@ int twoway_merge(struct cache_entry **src,
* stage0 does not have anything there. * stage0 does not have anything there.
*/ */
int bind_merge(struct cache_entry **src, int bind_merge(struct cache_entry **src,
struct unpack_trees_options *o) struct unpack_trees_options *o,
int remove)
{ {
struct cache_entry *old = src[0]; struct cache_entry *old = src[0];
struct cache_entry *a = src[1]; struct cache_entry *a = src[1];
remove_entry(remove);
if (o->merge_size != 1) if (o->merge_size != 1)
return error("Cannot do a bind merge of %d trees\n", return error("Cannot do a bind merge of %d trees\n",
o->merge_size); o->merge_size);
@ -891,11 +906,13 @@ int bind_merge(struct cache_entry **src,
* - take the stat information from stage0, take the data from stage1 * - take the stat information from stage0, take the data from stage1
*/ */
int oneway_merge(struct cache_entry **src, int oneway_merge(struct cache_entry **src,
struct unpack_trees_options *o) struct unpack_trees_options *o,
int remove)
{ {
struct cache_entry *old = src[0]; struct cache_entry *old = src[0];
struct cache_entry *a = src[1]; struct cache_entry *a = src[1];
remove_entry(remove);
if (o->merge_size != 1) if (o->merge_size != 1)
return error("Cannot do a oneway merge of %d trees", return error("Cannot do a oneway merge of %d trees",
o->merge_size); o->merge_size);

11
unpack-trees.h
View File

@ -4,7 +4,8 @@
struct unpack_trees_options; struct unpack_trees_options;
typedef int (*merge_fn_t)(struct cache_entry **src, typedef int (*merge_fn_t)(struct cache_entry **src,
struct unpack_trees_options *options); struct unpack_trees_options *options,
int remove);
struct unpack_trees_options { struct unpack_trees_options {
int reset; int reset;
@ -29,9 +30,9 @@ struct unpack_trees_options {
extern int unpack_trees(unsigned n, struct tree_desc *t, extern int unpack_trees(unsigned n, struct tree_desc *t,
struct unpack_trees_options *options); struct unpack_trees_options *options);
int threeway_merge(struct cache_entry **stages, struct unpack_trees_options *o); int threeway_merge(struct cache_entry **stages, struct unpack_trees_options *o, int);
int twoway_merge(struct cache_entry **src, struct unpack_trees_options *o); int twoway_merge(struct cache_entry **src, struct unpack_trees_options *o, int);
int bind_merge(struct cache_entry **src, struct unpack_trees_options *o); int bind_merge(struct cache_entry **src, struct unpack_trees_options *o, int);
int oneway_merge(struct cache_entry **src, struct unpack_trees_options *o); int oneway_merge(struct cache_entry **src, struct unpack_trees_options *o, int);
#endif #endif