Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx> writes:
> diff --git a/diff.c b/diff.c
> index afefe08..4b2029c 100644
> --- a/diff.c
> +++ b/diff.c
> @@ -1773,19 +1773,17 @@ int diff_populate_filespec(struct diff_filespec *s, int size_only)
> s->size = xsize_t(st.st_size);
> if (!s->size)
> goto empty;
> - if (size_only)
> - return 0;
> if (S_ISLNK(st.st_mode)) {
> ...
> }
> + if (size_only)
> + return 0;
It is unfortunate that we need to always readlink even when we only would
want to cull differences early (e.g. --raw without any fancy filters such
as rename detection), but symbolic links should be minorities in any sane
repo, and it should not be worth trying to optimize this for sane
filesystems by making it conditional.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
