From: "Marco Costalba" <mcostalba@gmail.com>
To: "Linus Torvalds" <torvalds@linux-foundation.org>
Cc: "Git Mailing List" <git@vger.kernel.org>,
"Paul Mackerras" <paulus@samba.org>
Subject: Re: [RFD] Strange patch formats (aka tricks with unified diffs)..
Date: Tue, 17 Apr 2007 20:02:30 +0200 [thread overview]
Message-ID: <e5bfff550704171102j61234553qb388a4be67592a07@mail.gmail.com> (raw)
In-Reply-To: <e5bfff550704122238l4a453cb7g38834c7da3432d5a@mail.gmail.com>
On 4/13/07, Marco Costalba <mcostalba@gmail.com> wrote:
> On 4/12/07, Linus Torvalds <torvalds@linux-foundation.org> wrote:
> >
> >
> > So I can see two ways to fix this:
> > - bug Paul about alternate diff viewing capabilities in gitk. He's cc'd
> > here. A way to make the "diff" pane show just the new one, the old one,
> > or even both side-by-side with some mousy interface?
> > - add some actual switch to git diff generation to hide the negative side
> > of a unified diff, and add some way to just make gitk pass that switch
> > in.
> >
> > I dunno. I realize that it's not just gitk - gitweb, qgit etc don't allow
> > the tricks *either*, but gitk was the one I just hit this with.
>
> Currently in qgit you can select a revision + a file and call an
> external diff viewer, as example Kompare (tweakable from
> 'Edir->settings->External diff viewer' menu) to view the diff
> side-by-side, or in any other way the viewer allows, for the selected
> file.
>
> I like also the idea to hide the negative side but I would rather
> prefer a togglable command/button instead of a command line argument
> or a setting, so to allow easy and quick hide/unhide on the same
> patch.
>
> I will work on it..... ;-)
>
I have pushed a patch with support for toggling view of the negative
side of an unified diff (right most 'filter' button in patch viewer
tab or CTRL+H shortcut).
In case someone is interested repo is, as usual,
git://git.kernel.org/pub/scm/qgit/qgit.git
Marco
P.S: Another two useful shortcuts are the hide/unhide of revision
description header ('h' key or from 'View' menu) and hide/unhide of
secondary panes ('s' key or from 'View' menu) they work in all the
different tabs.
next prev parent reply other threads:[~2007-04-17 18:02 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-04-12 21:58 [RFD] Strange patch formats (aka tricks with unified diffs) Linus Torvalds
2007-04-12 23:02 ` Junio C Hamano
2007-04-13 5:38 ` Marco Costalba
2007-04-17 18:02 ` Marco Costalba [this message]
2007-04-13 5:43 ` Paul Mackerras
2007-04-13 6:07 ` Martin Langhoff
2007-04-13 6:35 ` Junio C Hamano
2007-04-13 7:01 ` Martin Langhoff
2007-04-14 8:59 ` Johannes Schindelin
2007-04-14 9:55 ` Junio C Hamano
2007-04-14 10:09 ` Jeff King
2007-04-13 15:16 ` Linus Torvalds
2007-04-18 1:44 ` Paul Mackerras
2007-04-18 4:02 ` Linus Torvalds
2007-04-13 8:10 ` Matthias Lederhofer
2007-04-13 15:00 ` Linus Torvalds
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=e5bfff550704171102j61234553qb388a4be67592a07@mail.gmail.com \
--to=mcostalba@gmail.com \
--cc=git@vger.kernel.org \
--cc=paulus@samba.org \
--cc=torvalds@linux-foundation.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox