Thread (1 message) 1 message, 1 author, 2017-07-31

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

From: Junio C Hamano <hidden>
Date: 2017-07-31 16:32:32

Jeff King [off-list ref] 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 in commit_pager_choice(). So it's probably worth documenting
the "-1" behavior as you did here as a possible value for "def".
Thanks for reading it over. I agree that the "punt" behaviour is a
sensible one in that use case, and I also agree that it would be
good to explain it.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help