Re: [PATCH gitweb] Visually indicating patch size with horizontal bars

From: Petr Baudis <pasky@suse.cz>
Date: Nov 1, 2005, 23:30 UTC
Message-ID: <20051101233035.GB1431@pasky.or.cz>
In-Reply-To: <20051028023833.GA19939@pe.Belkin>

Dear diary, on Fri, Oct 28, 2005 at 04:38:33AM CEST, I got a letter where Chris Shoemaker <c.shoemaker@cox.net> told me that...

Show 13 quoted lines

> Here's a side-by-side comparison.  Open two browser tabs and flip between them:
> 
> http://www.codesifter.com/cgi-bin/gitweb-difftreeP.cgi?p=git.git;a=shortlog
> http://www.codesifter.com/cgi-bin/gitweb-difftreeNames.cgi?p=git.git;a=shortlog
> 
> I've used a project you all are familar with, and that has more than
> two files.  The first page uses 'git-diff-tree -p $hash|wc -l'.  The
> second page uses 'git-diff-tree -r --name-only|wc -l'.  (Oh and I have
> a merge indicator now.)
> 
> How do they compare for showing damage-potential?  I think they both
> do a reasonable job.  I think the full patch diff is a bit better, but
> it does cost.

What about having the color indicate the number of affected files (let's say on a blue..red scale) and the width the size of patch?

-- 
				Petr "Pasky" Baudis
Stuff: http://pasky.or.cz/
VI has two modes: the one in which it beeps and the one in which
it doesn't.