Re: [PATCH v2 7/8] name-hash: allow dir hashing even when !ignore_case
From: Eric Sunshine <hidden>
Date: 2016-06-15 22:59:58
On Sat, Feb 22, 2014 at 4:17 AM, Thomas Rast [off-list ref] wrote:
quoted hunk ↗ jump to hunk
The directory hash (for fast checks if the index already has a directory) was only used in ignore_case mode and so depended on that flag. Make it generally available on request. Signed-off-by: Thomas Rast <redacted> ---diff --git a/name-hash.c b/name-hash.c index e5b6e1a..c8953be 100644 --- a/name-hash.c +++ b/name-hash.c@@ -141,16 +141,19 @@ static void hash_index_entry(struct index_state *istate, struct cache_entry *ce) *pos = ce; } - if (ignore_case && !(ce->ce_flags & CE_UNHASHED)) + if (istate->has_dir_hash && !(ce->ce_flags & CE_UNHASHED)) add_dir_entry(istate, ce); } -static void lazy_init_name_hash(struct index_state *istate) +void init_name_hash(struct index_state *istate, int force_dir_hash) { int nr; if (istate->name_hash_initialized) return; + + istate->has_dir_hash = force_dir_hash || ignore_case;
This is getting a bit convoluted. Refactoring lazy_init_name_hash()
into two functions would eliminate the complexity. For instance:
void init_name_hash(struct index_state *istate)
{
...pure initialization code...
}
static void init_name_hash_if_needed(struct index_state *istate)
{
if (ignore_case && !istate->name_hash_initialized)
init_name_hash(istate);
}
The two existing callers of lazy_init_name_hash() in name-hash.c,
which rely upon the lazy/ignore-case logic, would invoke the static
init_name_hash_if_needed().
The new caller in patch 8/8, which knows explicitly if and when it
wants the hash initialized can invoke the public init_name_hash().
quoted hunk ↗ jump to hunk
if (istate->cache_nr) preallocate_hash(&istate->name_hash, istate->cache_nr); for (nr = 0; nr < istate->cache_nr; nr++)@@ -161,7 +164,7 @@ static void lazy_init_name_hash(struct index_state *istate) void add_name_hash(struct index_state *istate, struct cache_entry *ce) { /* if already hashed, add reference to directory entries */ - if (ignore_case && (ce->ce_flags & CE_STATE_MASK) == CE_STATE_MASK) + if (istate->has_dir_hash && (ce->ce_flags & CE_STATE_MASK) == CE_STATE_MASK) add_dir_entry(istate, ce); ce->ce_flags &= ~CE_UNHASHED;@@ -181,7 +184,7 @@ void add_name_hash(struct index_state *istate, struct cache_entry *ce) void remove_name_hash(struct index_state *istate, struct cache_entry *ce) { /* if already hashed, release reference to directory entries */ - if (ignore_case && (ce->ce_flags & CE_STATE_MASK) == CE_HASHED) + if (istate->has_dir_hash && (ce->ce_flags & CE_STATE_MASK) == CE_HASHED) remove_dir_entry(istate, ce); ce->ce_flags |= CE_UNHASHED;@@ -228,7 +231,7 @@ struct cache_entry *index_dir_exists(struct index_state *istate, const char *nam struct cache_entry *ce; struct dir_entry *dir; - lazy_init_name_hash(istate); + init_name_hash(istate, 0); dir = find_dir_entry(istate, name, namelen); if (dir && dir->nr) return dir->ce;@@ -250,7 +253,7 @@ struct cache_entry *index_file_exists(struct index_state *istate, const char *na unsigned int hash = hash_name(name, namelen); struct cache_entry *ce; - lazy_init_name_hash(istate); + init_name_hash(istate, 0); ce = lookup_hash(hash, &istate->name_hash); while (ce) {@@ -286,9 +289,11 @@ void free_name_hash(struct index_state *istate) if (!istate->name_hash_initialized) return; istate->name_hash_initialized = 0; - if (ignore_case) + if (istate->has_dir_hash) { /* free directory entries */ for_each_hash(&istate->dir_hash, free_dir_entry, NULL); + istate->has_dir_hash = 0; + } free_hash(&istate->name_hash); free_hash(&istate->dir_hash); --1.9.0.313.g3d0a325