Subject: Re: [PATCH] Break down no-lstat() condition checks in verify_uptodate()

From: Clemens Buchacher <[email-masked]>
Date: Sat, 30 Jul 2011 18:22:59 +0200

On Sat, Jul 30, 2011 at 10:55:05AM +0700, Nguyễn Thái Ngọc Duy wrote:
>
> +	/*
> +	 * CE_VALID and CE_SKIP_WORKTREE cheat, we better check again
> +	 * if this entry is truely uptodate because this file may be
> +	 * overwritten.
> +	 */

s/truely/truly

> +	if ((ce->ce_flags & CE_VALID) || ce_skip_worktree(ce))
> +		; /* keep checking */
> +	else if (o->reset || ce_uptodate(ce))
>  		return 0;

Nice! Acked-by: Clemens Buchacher <[email-masked]>