Suppose untracked cache stores that in directory A we need to recurse
in A/B and A/C. Then A/B is removed. When read_directory() is executed
again, of course we detect that we only need to recurse in A/C when in
A, not A/B any more.

We need a way though to let the write phase know not to write A/B
down. Which is the purpose of this bit. We can't simply destroy A/B
when A is invalidated, because at that moment we don't know if A/B is
deleted or not.
---
 dir.c | 15 ++++++++++++++-
 dir.h |  1 +
 2 files changed, 15 insertions(+), 1 deletion(-)

diff --git a/dir.c b/dir.c
index 205f323..63fa960 100644
--- a/dir.c
+++ b/dir.c
@@ -591,6 +591,7 @@ static void do_invalidate_gitignore(struct 
untracked_cache_dir *dir)
        int i;
        dir->valid = 0;
        dir->untracked_nr = 0;
+       /* dir->recurse = 0; ? */
        for (i = 0; i < dir->dirs_nr; i++)
                do_invalidate_gitignore(dir->dirs[i]);
 }
@@ -605,9 +606,12 @@ static void invalidate_gitignore(struct untracked_cache 
*uc,
 static void invalidate_directory(struct untracked_cache *uc,
                                 struct untracked_cache_dir *dir)
 {
+       int i;
        uc->dir_invalidated++;
        dir->valid = 0;
        dir->untracked_nr = 0;
+       for (i = 0; i < dir->dirs_nr; i++)
+               dir->dirs[i]->recurse = 0;
 }
 
 static int add_excludes(const char *fname,
@@ -1581,6 +1585,10 @@ int read_cached_dir(struct cached_dir *cdir)
        }
        while (cdir->nr_dirs < cdir->untracked->dirs_nr) {
                struct untracked_cache_dir *d = 
cdir->untracked->dirs[cdir->nr_dirs];
+               if (!d->recurse) {
+                       cdir->nr_dirs++;
+                       continue;
+               }
                cdir->ucd = d;
                cdir->nr_dirs++;
                return 0;
@@ -1602,8 +1610,10 @@ static void close_cached_dir(struct cached_dir *cdir)
         * We have gone through this directory and found no untracked
         * entries. Set untracked_nr to zero to make it valid.
         */
-       if (cdir->untracked && !cdir->untracked->valid)
+       if (cdir->untracked) {
                cdir->untracked->valid = 1;
+               cdir->untracked->recurse = 1;
+       }
 }
 
 /*
@@ -1849,6 +1859,9 @@ static struct untracked_cache_dir 
*validate_untracked_cache(struct dir_struct *d
                hashcpy(dir->untracked->excludes_file_sha1,
                        dir->excludes_file_sha1);
        }
+
+       /* Make sure this directory is not dropped out at saving phase */
+       root->recurse = 1;
        return root;
 }
 
diff --git a/dir.h b/dir.h
index 8955945..5dde37b 100644
--- a/dir.h
+++ b/dir.h
@@ -108,6 +108,7 @@ struct untracked_cache_dir {
        /* null SHA-1 means this directory does not have .gitignore */
        unsigned char exclude_sha1[20];
        struct stat_data stat_data;
+       unsigned int recurse : 1;
        unsigned int check_only : 1;
        unsigned int valid : 1;
        unsigned int untracked_nr : 29;
-- 
1.9.1.346.ga2b5940

--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to