On Sun, Dec 03, 2006 at 08:42:47PM +0100, Johannes Schindelin wrote:
quoted text
>=20
> A move of a directory should find the entries in the index by
> searching for the name _including_ the slash. Otherwise, the
> directory can be shadowed by a file when it matches the prefix
> and is lexicographically smaller, e.g. "ab.c" shadows "ab/".

Thanks - seems to work now, and the existing tests are not broken.

quoted text
> Noticed by Sergey Vlasov.
>=20
> Signed-off-by: Johannes Schindelin <Johannes.Schindelin@gmx.de>
> ---
>=20
> 	Good catch. Thanks!
>=20
>  builtin-mv.c |   11 +++++++----
>  1 files changed, 7 insertions(+), 4 deletions(-)
>=20
> diff --git a/builtin-mv.c b/builtin-mv.c
> index 54dd3bf..d14a4a7 100644
> --- a/builtin-mv.c
> +++ b/builtin-mv.c
> @@ -146,21 +146,24 @@ int cmd_mv(int argc, const char **argv, const char =
*prefix)
quoted text
>  				&& lstat(dst, &st) =3D=3D 0)
>  			bad =3D "cannot move directory over file";
>  		else if (src_is_dir) {
> +			const char *src_w_slash =3D add_slash(src);
> +			int len_w_slash =3D length + 1;
>  			int first, last;
> =20
>  			modes[i] =3D WORKING_DIRECTORY;
> =20
> -			first =3D cache_name_pos(src, length);
> +			first =3D cache_name_pos(src_w_slash, len_w_slash);
>  			if (first >=3D 0)
> -				die ("Huh? %s/ is in index?", src);
> +				die ("Huh? %.*s is in index?",
> +						len_w_slash, src_w_slash);
> =20
>  			first =3D -1 - first;
>  			for (last =3D first; last < active_nr; last++) {
>  				const char *path =3D active_cache[last]->name;
> -				if (strncmp(path, src, length)
> -						|| path[length] !=3D '/')
> +				if (strncmp(path, src_w_slash, len_w_slash))
>  					break;
>  			}
> +			free((char *)src_w_slash);
> =20
>  			if (last - first < 1)
>  				bad =3D "source directory is empty";