Re: [PATCH] git.el: Diff only file at point by default

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



David Kågedal <davidk@xxxxxxxxxxxxxx> writes:

> Use prefix (C-u) to diff all marked files.
>
> Signed-off-by: David Kågedal <davidk@xxxxxxxxxxxxxx>
> ---
>  contrib/emacs/git.el |   62 +++++++++++++++++++++++++++++---------------------
>  1 files changed, 36 insertions(+), 26 deletions(-)
>
> Here is an updated patch that udpate the stage diff commands as
> well. It doesn't touch git-diff-file-merge-head since that already
> uses prefix arguments. Don't know if there is a solution to that.

I'd suggest to change git-diff-file-merge-head to act on the current
file too. This means that the behavior is consistent across all diff
functions, even if we lose the ability to do git-diff-file-merge-head on
all marked files. Later on we may want to use a different mechanism than
the prefix arg to toggle the marked file behavior for all diff functions
(something like the pcl-cvs toggle-marks function).

Also make sure to check the callers of the diff functions you are
changing; at least git-log-edit-diff got broken by your change, there
may be others.

-- 
Alexandre Julliard
julliard@xxxxxxxxxx
--
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

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux