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: > > > >> Add a kind of "next" link in the bottom part of navigation bar for > >> "commitdiff" view. > >> > >> For commitdiff between two commits: > >> (from: _commit_) > >> For commitdiff for one single parent commit: > >> (parent: _commit_) > >> For commitdiff for one merge commit > >> (merge: _commit_ _commit_ ...) > >> For commitdiff for root (parentless) commit > >> (initial) > >> 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 > > 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. -- Petr "Pasky" Baudis Stuff: http://pasky.or.cz/ #!/bin/perl -sp0777i<X+d*lMLa^*lN%0]dsXx++lMlN/dsM0<j]dsj $/=unpack('H*',$_);$_=`echo 16dio\U$k"SK$/SM$n\EsN0p[lN*1 lK[d2%Sa2/d0$^Ixp"|dc`;s/\W//g;$_=pack('H*',/((..)*)$/) - 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