Robin Rosenberg <robin.rosenb...@dewire.com> writes:

> diff --git a/read-cache.c b/read-cache.c
> index fda78bc..f7fe15d 100644
> --- a/read-cache.c
> +++ b/read-cache.c
> @@ -197,8 +197,9 @@ static int ce_match_stat_basic(struct cache_entry *ce, 
> struct stat *st)
>       }
>       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)
> -             changed |= CTIME_CHANGED;
> +     if ((trust_ctime || ((check_nonzero_stat&CHECK_NONZERO_STAT_CTIME) && 
> ce->ce_ctime.sec)))

One SP is required on each side of a binary operator; please have
one after check_nonzero_stat and after the & after it.

I wonder if we should lose the trust_ctime variable and use this
check_nonzero_stat bitset exclusively, provided that this were a
good direction to go?
--
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