Re: [PATCH] gitweb: add a setting to control the tabstop width
- From
Jakub Narebski <jnareb@gmail.com>
- Date
- Mar 4, 2008, 00:08 UTC
- Message-ID
- <200803040108.58243.jnareb@gmail.com>
- In-Reply-To
- <20080303225200.GA8145@hashpling.org>
Charles Bailey wrote:
Show 17 quoted lines
> On Mon, Mar 03, 2008 at 11:33:28PM +0100, Jakub Narebski wrote: >> Charles Bailey <charles@hashpling.org> writes: >>> >>> The untabify function seems the sensible place to make the change. As >>> untabify is called once per line from various different locations it >>> also makes sense to cache the result of the config lookup in a package >>> variable, though this makes the change slightly less neat. >> >> Since b201927 (gitweb: Read repo config using 'git config -z -l') >> repository config is cached in %config hash (per repository), so >> I don't think global / package variable $tabstop_width is really >> needed... > > Fair point, although we still save the cost of some 'is the config > variable overrideable and if so is it overriden' logic. Untabify is a > once per line call which is more frequesnt than most gitweb config > checking calls.
Good enough.
One think I'd worry about is interaction with mod_perl (or FastCGI), namely if $tabstop_width wouldn't get stale information. Perhaps writing
our $tabstop_width = undef;
as initializer would be enough.
-- Jakub Narebski Poland