From: Marco Costalba <mcostalba@gmail.com>
Date: Fri, 25 Aug 2006 21:33:45 GMT
Subject: Re: [PATCH 19/19] gitweb: Remove creating directory for temporary files
Message-ID: <e5bfff550608251433o36713ee1na5544992320b5845@mail.gmail.com>
In-Reply-To: <200608252135.27894.jnareb@gmail.com>

>
> You can view new gitweb in work at
>   http://front.fuw.edu.pl/cgi-bin/jnareb/gitweb.cgi
>

Very nice job!

Just a couple of suggestion regarding blame.

- Instead of commit sha perhaps is more useful to show author name
(linked to commit) and progressive number of revision.

- Original code lines, ie. imported at the beginning and never
modified, perhaps it is better to view without commit number, this
could obfuscate the view and in any case is not an accurate info
because the line was not modified during initial patch.
[Note] Also in qgit there was the annotation of the first initial
commit in case of unmodified code lines, then Ingo Molnar suggest me
to remove that line and leave it blank for the reasons I reported
above. I think he was right.

Marco