Re: [ANNOUNCE] tig-0.14

From: Jonas Fonseca <fonseca@diku.dk>
Date: Feb 6, 2009, 22:10 UTC
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:

Show 7 quoted lines

> 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.

Show 5 quoted lines

> 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