diff options
author | Carl Worth <cworth@cworth.org> | 2010-01-05 16:42:14 -0800 |
---|---|---|
committer | Carl Worth <cworth@cworth.org> | 2010-01-06 10:32:06 -0800 |
commit | 2a98b1d48795e66754435e06e28cdf58ab1dd154 (patch) | |
tree | 80c9638dbc9978eeec78d3a65b1898a4c55ab4c0 /notmuch-new.c | |
parent | 28ce73848d98d8ee2b661733402e2c10b13418d5 (diff) |
add_files_recursive: Make the maildir detection more efficient.
Previously, we were re-scanning the entire list of entries for every
directory entry. Instead, we can simply check if the entries look like
a maildir once, up-front.
Diffstat (limited to 'notmuch-new.c')
-rw-r--r-- | notmuch-new.c | 16 |
1 files changed, 10 insertions, 6 deletions
diff --git a/notmuch-new.c b/notmuch-new.c index f34d4676..af742977 100644 --- a/notmuch-new.c +++ b/notmuch-new.c @@ -100,12 +100,14 @@ static int ino_cmp(const struct dirent **a, const struct dirent **b) * Return 1 if the directory looks like a Maildir and 0 otherwise. */ static int -is_maildir (struct dirent **entries, int count) +_entries_resemble_maildir (struct dirent **entries, int count) { int i, found = 0; for (i = 0; i < count; i++) { - if (entries[i]->d_type != DT_DIR) continue; + if (entries[i]->d_type != DT_DIR) + continue; + if (strcmp(entries[i]->d_name, "new") == 0 || strcmp(entries[i]->d_name, "cur") == 0 || strcmp(entries[i]->d_name, "tmp") == 0) @@ -153,6 +155,7 @@ add_files_recursive (notmuch_database_t *notmuch, int i, num_fs_entries; notmuch_directory_t *directory; struct stat st; + notmuch_bool_t is_maildir; if (stat (path, &st)) { fprintf (stderr, "Error reading directory %s: %s\n", @@ -180,6 +183,8 @@ add_files_recursive (notmuch_database_t *notmuch, } /* First, recurse into all sub-directories. */ + is_maildir = _entries_resemble_maildir (fs_entries, num_fs_entries); + for (i = 0; i < num_fs_entries; i++) { if (interrupted) break; @@ -190,15 +195,14 @@ add_files_recursive (notmuch_database_t *notmuch, continue; /* Ignore special directories to avoid infinite recursion. - * Also ignore the .notmuch directory. + * Also ignore the .notmuch directory and any "tmp" directory + * that appears within a maildir. */ /* XXX: Eventually we'll want more sophistication to let the * user specify files to be ignored. */ if (strcmp (entry->d_name, ".") == 0 || strcmp (entry->d_name, "..") == 0 || - (entry->d_type == DT_DIR && - (strcmp (entry->d_name, "tmp") == 0) && - is_maildir (fs_entries, num_fs_entries)) || + (is_maildir && strcmp (entry->d_name, "tmp") == 0) || strcmp (entry->d_name, ".notmuch") ==0) { continue; |