Re: [PATCH v2 05/10] git.c: provide setup_auto_pager()

2017-07-31 Thread Junio C Hamano
Jeff King writes: > But thinking on it, the most plausible case is something like: > > setup_auto_pager("foo", -1); > ... > /* fallback to some historical compatibility name */ > setup_auto_pager("bar", 0); > > And it's important for the "-1" there to be a true punt, and not do > anything

Re: [PATCH v2 05/10] git.c: provide setup_auto_pager()

2017-07-30 Thread Jeff King
On Mon, Jul 17, 2017 at 10:10:47PM +0200, Martin Ågren wrote: > The previous patch introduced a way for builtins to declare that they > will take responsibility for handling the `pager.foo`-config item. (See > the commit message of that patch for why that could be useful.) > > Provide setup_auto_

[PATCH v2 05/10] git.c: provide setup_auto_pager()

2017-07-17 Thread Martin Ågren
The previous patch introduced a way for builtins to declare that they will take responsibility for handling the `pager.foo`-config item. (See the commit message of that patch for why that could be useful.) Provide setup_auto_pager(), which builtins can call in order to handle `pager.`, including p