Petr Baudis wrote: > Dear diary, on Tue, Oct 24, 2006 at 12:08:08AM CEST, I got a letter > where Jakub Narebski <jnareb@xxxxxxxxx> said that... >> Use fixed string instead of shortened SHA1 identifier of commit >> as a name for "mext" link in commitdiff view. >> >> Signed-off-by: Jakub Narebski <jnareb@xxxxxxxxx> > > So I've read what the patch actually does this time, and I really > dislike it. > >> Junio C Hamano wrote: >>> Jakub Narebski <jnareb@xxxxxxxxx> writes: [...] >>>> For commitdiff for one merge commit >>>> (merge: _commit_ _commit_ ...) [...] >>>> where _link_ denotes hyperlink. SHA1 is shortened to 7 characters on >>>> display, everything is perhaps unnecessary esc_html on display. >>> >>> I always hated gitweb diffs that prefix each filepair with their >>> full 40-byte SHA-1 blob object names. It just adds noise to the >>> output without adding any meaningful information. > > I agree, using the shortened SHA1 would be definitely an improvement > here, but I'm planning on revamping gitweb diff output. Any ideas? >>> Would it even be necessary to use any SHA-1 name in these cases, >>> I wonder. Would it make the page less useful if we replace all >>> of the above _commit_ with a fixed string, say, "parent"? > > I really disagree here - what's the point of not using SHA-1? The extra > string carries zero information in comparison with the previous state > and I just can't see how it *improves* stuff. If you're walking in a > maze and making marks on walls, it's still more useful if you have > corridors named by "A", "B", "C", "D" on junctions if you sometimes want > to walk back to the marked corridors. That's why instead of resending the patch/amending the commit, I have put changing shortened SHA1 to constant name in new patch. By the way, because those links to parents commitdiffs are just ordinary links, you can see when those links are visited. -- Jakub Narebski Poland - To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html