From: Jonas Fonseca <fonseca@diku.dk>
Date: Fri, 06 Feb 2009 22:10:23 GMT
Subject: Re: [ANNOUNCE] tig-0.14
Message-ID: <2c6b72b30902061410l64c98c33g19b97f656d347c83@mail.gmail.com>
In-Reply-To: <20090206191511.GD19494@coredump.intra.peff.net>

On Fri, Feb 6, 2009 at 20:15, Jeff King <peff@peff.net> wrote:
> On Thu, Feb 05, 2009 at 09:44:36PM +0100, Jonas Fonseca wrote:
>
>>  - Blame view: load blame for parent commit. For merge commits the parent
>>    is queried. Bound to ',' by default via the existing "parent" action.
>
> Thanks for this, btw. I've already used it at least half a dozen times
> in the past week or so.

Good to hear. I remember you posted a patch for this after 0.11 was
released last April.

> It looks like you just keep the view on the same line number when moving
> to the new blame output. In practice, this has very mixed results. Most
> of the time it does exactly what I want, but if the file changes
> significantly, you get dumped at a totally unrelated part of the file.
> I'm not sure if there is a more clever solution, though.

Yes, it is a bit easy to get lost. It should be possible to find the
original line number either by making git-blame also honor
--show-number for the --incremental output or by using the "porcelain"
version:

  git blame --show-number -L <line>,<line> <rev> <file>

-- 
Jonas Fonseca