Re: [PATCH] gitweb: Adding a `blame' interface.

From: Linus Torvalds <torvalds@osdl.org>
Date: Jun 12, 2006, 23:21 UTC
Message-ID: <Pine.LNX.4.64.0606121619510.5498@g5.osdl.org>
In-Reply-To: <Pine.LNX.4.63.0606130053400.25682@wbgn013.biozentrum.uni-wuerzburg.de>

On Tue, 13 Jun 2006, Johannes Schindelin wrote:

Show 9 quoted lines

> > 
> > The war _I_d like to see is the GUI thing which does the "show when this 
> > section changed last" by following the history down only so far that the 
> > selected section shows up in the diff against the most current thing.
> 
> But this is just the next step! Nothing prevents you -- once everybody 
> agrees that blame/annotate does the right thing -- to restrict the lines 
> of interest. And AFAICT both blame and annotate are good at stopping when 
> all lines are accounted for.

You misunderstand. It's not "all", it's "any" (in fact, it would be even better if it would be on a byte-range basis, not on a line-based diff basis).

And it needs the GUI to make it useful, because nobody in their right mind will say "git showchange fs/inode.c 77-89" (and then do it iteratively if the first one wasn't actually the interesting case).

		Linus