Re: [RFC/PATCH 2/2] gitweb: Add an option to show size of blobs in the tree view
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Jul 31, 2007, 22:12 UTC
- Message-ID
- <7vd4y8fcjw.fsf@assigned-by-dhcp.cox.net>
- In-Reply-To
- <11858807944170-git-send-email-jnareb@gmail.com>
Jakub Narebski <jnareb@gmail.com> writes:
Show 10 quoted lines
> It allows to play with 'tree' view with blob size. Currently you > have to start browsing by adding ";opt=-l" to the gitweb URL by > hand. There is not link which will change view from ordinary 'tree' > view to 'tree' view with blob sizes. > > The 'tree' with blob size view is slightly more costly than the > ordinary, old 'tree' view, but not much more (0.018s vs 0.012s > in the hot cache case), so I don't think we need to control it > as a enabled (or disabled) feature, overrideable or not. It > probably should not be default.
I do not think there is any reason to forbid its use, as the "-l" to ls-tree was introduced for exactly this purpose, However, it might make sense to make the use of -l optional via the %feature mechanism. 50% increase even on hot cache case is not a price people who run busy sites would want to pay.