Christian Couder <christian.couder@xxxxxxxxx> writes: > Signed-off-by: Christian Couder <chriscool@xxxxxxxxxxxxx> > --- > dir.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/dir.c b/dir.c > index ffc0286..3b83cc0 100644 > --- a/dir.c > +++ b/dir.c > @@ -1954,6 +1954,7 @@ void add_untracked_cache(void) > > void remove_untracked_cache(void) > { > + free_untracked_cache(the_index.untracked); > the_index.untracked = NULL; > the_index.cache_changed |= UNTRACKED_CHANGED; > } Up to this point the series makes sense (again, I am not saying the remainder does not ;-)). But shouldn't this step, as a bugfix, appear a lot earlier in the series? -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html