status: always report ignored tracked directories

When enumerating paths that are ignored, paths the index knows about are not included in the result. The "index knows about" check is done by consulting the name hash, not the actual contents of the index: - When core.ignorecase is false, directory names are not in the name hash, and ignored ones are shown as ignored (directories can never be tracked anyway). - When core.ignorecase is true, however, the name hash keeps track of the names of directories, in order to detect additions of the paths under different cases. This causes ignored directories to be mistakenly excluded when enumerating ignored paths. Stop excluding directories that are in the name hash when looking for ignored files in dir_add_name(); the names that are actually in the index are excluded much earlier in the callchain in treat_file(), so this fix will not make them mistakenly identified as ignored. Signed-off-by: Antoine Pelisse <apelisse@gmail.com> Reviewed-by: Jeff King <peff@peff.net> Signed-off-by: Junio C Hamano <gitster@pobox.com>

Antoine Pelisse committed Jan 6, 2013 at 23:09 UTC a45fb697f13bb789aca930ce786cf416ef1ecd0e
1 file changed +3 -6
dir.c
+3 -6
@@ -672,7 +672,8 @@ static struct dir_entry *dir_entry_new(const char *pathname, int len)
672
673 static struct dir_entry *dir_add_name(struct dir_struct *dir, const char *pathname, int len)
674 {
675 - if (cache_name_exists(pathname, len, ignore_case))
675 + if (!(dir->flags & DIR_SHOW_IGNORED) &&
676 + cache_name_exists(pathname, len, ignore_case))
677 return NULL;
678
679 ALLOC_GROW(dir->entries, dir->nr+1, dir->alloc);
@@ -877,11 +878,7 @@ static int treat_file(struct dir_struct *dir, struct strbuf *path, int exclude,
878 if (exclude)
879 exclude_file = !(dir->flags & DIR_SHOW_IGNORED);
880 else if (dir->flags & DIR_SHOW_IGNORED) {
880 - /*
881 - * Optimization:
882 - * Don't spend time on indexed files, they won't be
883 - * added to the list anyway
884 - */
881 + /* Always exclude indexed files */
882 struct cache_entry *ce = index_name_exists(&the_index,
883 path->buf, path->len, ignore_case);
884