diff options
author | Linus Torvalds <torvalds@linux-foundation.org> | 2007-03-30 20:39:30 -0700 |
---|---|---|
committer | Junio C Hamano <junkio@cox.net> | 2007-03-31 17:41:32 -0700 |
commit | 9fc42d609197ef331f1c5b2ce5ef6fc9f2ee634f (patch) | |
tree | b59890e73a673167a4db90d85782851464c758f1 /dir.h | |
parent | d8b6a1a10b93666246984a50d64a163e71163aeb (diff) | |
download | git-9fc42d609197ef331f1c5b2ce5ef6fc9f2ee634f.tar.gz git-9fc42d609197ef331f1c5b2ce5ef6fc9f2ee634f.tar.xz |
Optimize directory listing with pathspec limiter.
The way things are set up, you can now pass a "pathspec" to the
"read_directory()" function. If you pass NULL, it acts exactly
like it used to do (read everything). If you pass a non-NULL
pointer, it will simplify it into a "these are the prefixes
without any special characters", and stop any readdir() early if
the path in question doesn't match any of the prefixes.
NOTE! This does *not* obviate the need for the caller to do the *exact*
pathspec match later. It's a first-level filter on "read_directory()", but
it does not do the full pathspec thing. Maybe it should. But in the
meantime, builtin-add.c really does need to do first
read_directory(dir, .., pathspec);
if (pathspec)
prune_directory(dir, pathspec, baselen);
ie the "prune_directory()" part will do the *exact* pathspec pruning,
while the "read_directory()" will use the pathspec just to do some quick
high-level pruning of the directories it will recurse into.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Junio C Hamano <junkio@cox.net>
Diffstat (limited to 'dir.h')
-rw-r--r-- | dir.h | 2 |
1 files changed, 1 insertions, 1 deletions
@@ -48,7 +48,7 @@ extern int common_prefix(const char **pathspec); #define MATCHED_EXACTLY 3 extern int match_pathspec(const char **pathspec, const char *name, int namelen, int prefix, char *seen); -extern int read_directory(struct dir_struct *, const char *path, const char *base, int baselen); +extern int read_directory(struct dir_struct *, const char *path, const char *base, int baselen, const char **pathspec); extern int push_exclude_per_directory(struct dir_struct *, const char *, int); extern void pop_exclude_per_directory(struct dir_struct *, int); |