Jeff King <peff@xxxxxxxx> writes:
> diff --git a/builtin/diff.c b/builtin/diff.c
> index d184aafab..8ed1e99e3 100644
> --- a/builtin/diff.c
> +++ b/builtin/diff.c
> @@ -409,7 +409,7 @@ int cmd_diff(int argc, const char **argv, const char *prefix)
> if (2 <= blobs)
> die(_("more than two blobs given: '%s'"), name);
> hashcpy(blob[blobs].oid.hash, obj->oid.hash);
> - blob[blobs].name = name;
> + blob[blobs].name = entry->path ? entry->path : name;
> blob[blobs].mode = entry->mode;
> blobs++;
> ...
> - add_pending_object(revs, a_obj, this);
> - add_pending_object(revs, b_obj, next);
> + add_pending_object_with_path(revs, a_obj, this,
> + oc.mode,
> + oc.path[0] ? oc.path : NULL);
> + add_pending_object_with_path(revs, b_obj, next,
> + oc2.mode,
> + oc2.path[0] ? oc2.path : NULL);
The fix is surprisingly simple, and I think it definitely goes in
the right direction.
Somehow, it bothers me to be forced to view (a_obj, this, from_sha1,
oc) vs (b_obj, next, sha1, oc2) as a sensibly corresponding pair of
tuples, but that is not something your introduction of "oc2"
started, so I won't complain ;-).
> return 0;
> }
> *dotdot = '.';
> @@ -1574,7 +1578,7 @@ int handle_revision_arg(const char *arg_, struct rev_info *revs, int flags, unsi
> verify_non_filename(revs->prefix, arg);
> object = get_reference(revs, arg, sha1, flags ^ local_flags);
> add_rev_cmdline(revs, object, arg_, REV_CMD_REV, flags ^ local_flags);
> - add_pending_object_with_mode(revs, object, arg, oc.mode);
> + add_pending_object_with_path(revs, object, arg, oc.mode, oc.path);
> return 0;
> }
>