"Jeff Hostetler via GitGitGadget" <gitgitgadget@xxxxxxxxx> writes: > Here is version 3. I think I have addressed the remaining comments. > > I cleaned up the test code to use the test_expect_failure at the beginning > and squashed in the test_expect_success version of tests into the final > commit in the series. > > I moved the invalidate_ce_fsm() commit earlier in the series, so that the > final commit actually uses all of the up-to-this-point changes to fix the > problem. > > I converted a few "should not happens" to BUG()s. > > Thanks to everyone for their time and attention reviewing this. Jeff > > Jeff Hostetler (14): > name-hash: add index_dir_find() > t7527: add case-insensitve test for FSMonitor > fsmonitor: refactor refresh callback on directory events > fsmonitor: clarify handling of directory events in callback helper > fsmonitor: refactor refresh callback for non-directory events > dir: create untracked_cache_invalidate_trimmed_path() > fsmonitor: refactor untracked-cache invalidation > fsmonitor: move untracked-cache invalidation into helper functions > fsmonitor: return invalidated cache-entry count on directory event > fsmonitor: remove custom loop from non-directory path handler > fsmonitor: return invalided cache-entry count on non-directory event > fsmonitor: trace the new invalidated cache-entry count > fsmonitor: refactor bit invalidation in refresh callback > fsmonitor: support case-insensitive events > > dir.c | 20 +++ > dir.h | 7 + > fsmonitor.c | 312 +++++++++++++++++++++++++++++------ > name-hash.c | 9 +- > name-hash.h | 7 +- > t/t7527-builtin-fsmonitor.sh | 223 +++++++++++++++++++++++++ > 6 files changed, 522 insertions(+), 56 deletions(-) Much nicer. Will queue. Thanks.