diff options
| author | Junio C Hamano <gitster@pobox.com> | 2011-10-05 12:35:53 -0700 | 
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2011-10-05 12:35:53 -0700 | 
| commit | 1b840a56629c8c0620c261672cc5d4d35ba253fa (patch) | |
| tree | 38691399a0fc7bb3bc5eac7f5e8b80dadbad573a | |
| parent | 2c4610393195cc6f35ea79efadbc5c7fd7470891 (diff) | |
| parent | 2f88c19700feb8db8f116f94bf558e61c82d543c (diff) | |
| download | git-1b840a56629c8c0620c261672cc5d4d35ba253fa.tar.gz | |
Merge branch 'jc/diff-index-unpack'
* jc/diff-index-unpack:
  diff-index: pass pathspec down to unpack-trees machinery
  unpack-trees: allow pruning with pathspec
  traverse_trees(): allow pruning with pathspec
| -rw-r--r-- | diff-lib.c | 1 | ||||
| -rw-r--r-- | tree-walk.c | 39 | ||||
| -rw-r--r-- | tree-walk.h | 1 | ||||
| -rw-r--r-- | unpack-trees.c | 2 | ||||
| -rw-r--r-- | unpack-trees.h | 1 | 
5 files changed, 38 insertions, 6 deletions
| diff --git a/diff-lib.c b/diff-lib.c index f8454dd291..ebe751e72d 100644 --- a/diff-lib.c +++ b/diff-lib.c @@ -468,6 +468,7 @@ static int diff_cache(struct rev_info *revs,  	opts.unpack_data = revs;  	opts.src_index = &the_index;  	opts.dst_index = NULL; +	opts.pathspec = &revs->diffopt.pathspec;  	init_tree_desc(&t, tree->buffer, tree->size);  	return unpack_trees(1, &t, &opts); diff --git a/tree-walk.c b/tree-walk.c index 33f749e1e7..808bb55ba3 100644 --- a/tree-walk.c +++ b/tree-walk.c @@ -309,6 +309,18 @@ static void free_extended_entry(struct tree_desc_x *t)  	}  } +static inline int prune_traversal(struct name_entry *e, +				  struct traverse_info *info, +				  struct strbuf *base, +				  int still_interesting) +{ +	if (!info->pathspec || still_interesting == 2) +		return 2; +	if (still_interesting < 0) +		return still_interesting; +	return tree_entry_interesting(e, base, 0, info->pathspec); +} +  int traverse_trees(int n, struct tree_desc *t, struct traverse_info *info)  {  	int ret = 0; @@ -316,10 +328,18 @@ int traverse_trees(int n, struct tree_desc *t, struct traverse_info *info)  	struct name_entry *entry = xmalloc(n*sizeof(*entry));  	int i;  	struct tree_desc_x *tx = xcalloc(n, sizeof(*tx)); +	struct strbuf base = STRBUF_INIT; +	int interesting = 1;  	for (i = 0; i < n; i++)  		tx[i].d = t[i]; +	if (info->prev) { +		strbuf_grow(&base, info->pathlen); +		make_traverse_path(base.buf, info->prev, &info->name); +		base.buf[info->pathlen-1] = '/'; +		strbuf_setlen(&base, info->pathlen); +	}  	for (;;) {  		unsigned long mask, dirmask;  		const char *first = NULL; @@ -376,16 +396,22 @@ int traverse_trees(int n, struct tree_desc *t, struct traverse_info *info)  			mask |= 1ul << i;  			if (S_ISDIR(entry[i].mode))  				dirmask |= 1ul << i; +			e = &entry[i];  		}  		if (!mask)  			break; -		ret = info->fn(n, mask, dirmask, entry, info); -		if (ret < 0) { -			error = ret; -			if (!info->show_all_errors) -				break; +		interesting = prune_traversal(e, info, &base, interesting); +		if (interesting < 0) +			break; +		if (interesting) { +			ret = info->fn(n, mask, dirmask, entry, info); +			if (ret < 0) { +				error = ret; +				if (!info->show_all_errors) +					break; +			} +			mask &= ret;  		} -		mask &= ret;  		ret = 0;  		for (i = 0; i < n; i++)  			if (mask & (1ul << i)) @@ -395,6 +421,7 @@ int traverse_trees(int n, struct tree_desc *t, struct traverse_info *info)  	for (i = 0; i < n; i++)  		free_extended_entry(tx + i);  	free(tx); +	strbuf_release(&base);  	return error;  } diff --git a/tree-walk.h b/tree-walk.h index 39524b7dba..0089581e1d 100644 --- a/tree-walk.h +++ b/tree-walk.h @@ -44,6 +44,7 @@ struct traverse_info {  	struct traverse_info *prev;  	struct name_entry name;  	int pathlen; +	struct pathspec *pathspec;  	unsigned long conflicts;  	traverse_callback_t fn; diff --git a/unpack-trees.c b/unpack-trees.c index cc616c3f99..670b464738 100644 --- a/unpack-trees.c +++ b/unpack-trees.c @@ -444,6 +444,7 @@ static int traverse_trees_recursive(int n, unsigned long dirmask,  	newinfo = *info;  	newinfo.prev = info; +	newinfo.pathspec = info->pathspec;  	newinfo.name = *p;  	newinfo.pathlen += tree_entry_len(p->path, p->sha1) + 1;  	newinfo.conflicts |= df_conflicts; @@ -1040,6 +1041,7 @@ int unpack_trees(unsigned len, struct tree_desc *t, struct unpack_trees_options  		info.fn = unpack_callback;  		info.data = o;  		info.show_all_errors = o->show_all_errors; +		info.pathspec = o->pathspec;  		if (o->prefix) {  			/* diff --git a/unpack-trees.h b/unpack-trees.h index 7998948307..5e432f576e 100644 --- a/unpack-trees.h +++ b/unpack-trees.h @@ -52,6 +52,7 @@ struct unpack_trees_options {  	const char *prefix;  	int cache_bottom;  	struct dir_struct *dir; +	struct pathspec *pathspec;  	merge_fn_t fn;  	const char *msgs[NB_UNPACK_TREES_ERROR_TYPES];  	/* | 
