unpack-trees: introduce preserve_ignored to unpack_trees_options
Currently, every caller of unpack_trees() that wants to ensure ignored files are overwritten by default needs to: * allocate unpack_trees_options.dir * flip the DIR_SHOW_IGNORED flag in unpack_trees_options.dir->flags * call setup_standard_excludes AND then after the call to unpack_trees() needs to * call dir_clear() * deallocate unpack_trees_options.dir That's a fair amount of boilerplate, and every caller uses identical code. Make this easier by instead introducing a new boolean value where the default value (0) does what we want so that new callers of unpack_trees() automatically get the appropriate behavior. And move all the handling of unpack_trees_options.dir into unpack_trees() itself. While preserve_ignored = 0 is the behavior we feel is the appropriate default, we defer fixing commands to use the appropriate default until a later commit. So, this commit introduces several locations where we manually set preserve_ignored=1. This makes it clear where code paths were previously preserving ignored files when they should not have been; a future commit will flip these to instead use a value of 0 to get the behavior we want. Signed-off-by: Elijah Newren <newren@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>
This commit is contained in:

committed by
Junio C Hamano

parent
491a7575f1
commit
04988c8d18
8
merge.c
8
merge.c
@ -53,7 +53,6 @@ int checkout_fast_forward(struct repository *r,
|
||||
struct unpack_trees_options opts;
|
||||
struct tree_desc t[MAX_UNPACK_TREES];
|
||||
int i, nr_trees = 0;
|
||||
struct dir_struct dir = DIR_INIT;
|
||||
struct lock_file lock_file = LOCK_INIT;
|
||||
|
||||
refresh_index(r->index, REFRESH_QUIET, NULL, NULL, NULL);
|
||||
@ -80,11 +79,7 @@ int checkout_fast_forward(struct repository *r,
|
||||
}
|
||||
|
||||
memset(&opts, 0, sizeof(opts));
|
||||
if (overwrite_ignore) {
|
||||
dir.flags |= DIR_SHOW_IGNORED;
|
||||
setup_standard_excludes(&dir);
|
||||
opts.dir = &dir;
|
||||
}
|
||||
opts.preserve_ignored = !overwrite_ignore;
|
||||
|
||||
opts.head_idx = 1;
|
||||
opts.src_index = r->index;
|
||||
@ -101,7 +96,6 @@ int checkout_fast_forward(struct repository *r,
|
||||
clear_unpack_trees_porcelain(&opts);
|
||||
return -1;
|
||||
}
|
||||
dir_clear(&dir);
|
||||
clear_unpack_trees_porcelain(&opts);
|
||||
|
||||
if (write_locked_index(r->index, &lock_file, COMMIT_LOCK))
|
||||
|
Reference in New Issue
Block a user