git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH 1/3] completion: fix alias listings with newlines

From
Shawn O. Pearce <spearce@spearce.org>
Date
Oct 8, 2009, 04:29 UTC
Message-ID
<20091008042917.GX9261@spearce.org>
In-Reply-To
<4ACCE417.5080907@gmail.com>
Stephen Boyd <bebarino@gmail.com> wrote:
> Johannes Sixt wrote:
> > Is it necessary to change the body of the loop? Your version spawns two
> > processes on each iteration, while the original spawned no processes.

Yes, we should try to avoid spawning a process here, it fires on each completion attempt if I recall.

> Maybe a better solution would be to add --keys-only or something to
> git-config?

Or a format string with a language based escape like for-each-ref supports? Might make it easier to use git config in scripts if we can write things like:

  git config --format='$data{%(key)}=%(value);' --perl
-- 
Shawn.
Previous: Stephen BoydNext: Stephen Boyd
Message 8 of 15 in “completion: fix alias listings with newlines”
  1. 1/3 completion: fix alias listings with newlinesStephen Boyd, Oct 7, 2009
  2. 2/3 completion: update am, commit, and logStephen Boyd, Oct 7, 2009
  3. 3/3 completion: add dirstat and friends to diff optionsStephen Boyd, Oct 7, 2009
  4. Junio C HamanoOct 7, 2009
  5. Junio C HamanoOct 7, 2009
  6. Johannes SixtOct 7, 2009
  7. Stephen BoydOct 7, 2009
  8. Shawn O. PearceOct 8, 2009
  9. Stephen BoydOct 8, 2009
  10. Johannes SixtOct 8, 2009
  11. Stephen BoydOct 8, 2009
  12. Johannes SixtOct 8, 2009
  13. Junio C HamanoOct 7, 2009
  14. Shawn O. PearceOct 8, 2009
  15. Stephen BoydOct 8, 2009

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.