Modify match_stat_basic, into one function that handles the
general case, which is the same for all index formats, and
a function that handles the specific parts for each index
file version.
Signed-off-by: Thomas Gummerer <t.gumme...@gmail.com>
---
read-cache.c | 71 +++++++++++++++++++++++++++++++++-------------------------
1 file changed, 40 insertions(+), 31 deletions(-)
diff --git a/read-cache.c b/read-cache.c
index 24b5e02..6e8991a 100644
--- a/read-cache.c
+++ b/read-cache.c
@@ -163,38 +163,10 @@ static int ce_modified_check_fs(struct cache_entry *ce,
struct stat *st)
return 0;
}
-static int ce_match_stat_basic(struct cache_entry *ce, struct stat *st)
+static int ce_match_stat_basic_v2(struct cache_entry *ce,
+ struct stat *st,
+ int changed)
{
- unsigned int changed = 0;
-
- if (ce->ce_flags & CE_REMOVE)
- return MODE_CHANGED | DATA_CHANGED | TYPE_CHANGED;
-
- switch (ce->ce_mode & S_IFMT) {
- case S_IFREG:
- changed |= !S_ISREG(st->st_mode) ? TYPE_CHANGED : 0;
- /* We consider only the owner x bit to be relevant for
- * "mode changes"
- */
- if (trust_executable_bit &&
- (0100 & (ce->ce_mode ^ st->st_mode)))
- changed |= MODE_CHANGED;
- break;
- case S_IFLNK:
- if (!S_ISLNK(st->st_mode) &&
- (has_symlinks || !S_ISREG(st->st_mode)))
- changed |= TYPE_CHANGED;
- break;
- case S_IFGITLINK:
- /* We ignore most of the st_xxx fields for gitlinks */
- if (!S_ISDIR(st->st_mode))
- changed |= TYPE_CHANGED;
- else if (ce_compare_gitlink(ce))
- changed |= DATA_CHANGED;
- return changed;
- default:
- die("internal error: ce_mode is %o", ce->ce_mode);
- }
if (ce->ce_mtime.sec != (unsigned int)st->st_mtime)
changed |= MTIME_CHANGED;
if (trust_ctime && ce->ce_ctime.sec != (unsigned int)st->st_ctime)
@@ -235,6 +207,43 @@ static int ce_match_stat_basic(struct cache_entry *ce,
struct stat *st)
return changed;
}
+static int ce_match_stat_basic(struct cache_entry *ce, struct stat *st)
+{
+ unsigned int changed = 0;
+
+ if (ce->ce_flags & CE_REMOVE)
+ return MODE_CHANGED | DATA_CHANGED | TYPE_CHANGED;
+
+ switch (ce->ce_mode & S_IFMT) {
+ case S_IFREG:
+ changed |= !S_ISREG(st->st_mode) ? TYPE_CHANGED : 0;
+ /* We consider only the owner x bit to be relevant for
+ * "mode changes"
+ */
+ if (trust_executable_bit &&
+ (0100 & (ce->ce_mode ^ st->st_mode)))
+ changed |= MODE_CHANGED;
+ break;
+ case S_IFLNK:
+ if (!S_ISLNK(st->st_mode) &&
+ (has_symlinks || !S_ISREG(st->st_mode)))
+ changed |= TYPE_CHANGED;
+ break;
+ case S_IFGITLINK:
+ /* We ignore most of the st_xxx fields for gitlinks */
+ if (!S_ISDIR(st->st_mode))
+ changed |= TYPE_CHANGED;
+ else if (ce_compare_gitlink(ce))
+ changed |= DATA_CHANGED;
+ return changed;
+ default:
+ die("internal error: ce_mode is %o", ce->ce_mode);
+ }
+
+ changed = ce_match_stat_basic_v2(ce, st, changed);
+ return changed;
+}
+
static int is_racy_timestamp(const struct index_state *istate, struct
cache_entry *ce)
{
return (!S_ISGITLINK(ce->ce_mode) &&
--
1.7.10.GIT
--
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
