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

Re: [PATCH v5] completion: add new __git_complete helper

From
Junio C Hamano <gitster@pobox.com>
Date
May 14, 2012, 19:09 UTC
Message-ID
<7vaa1ah95p.fsf@alter.siamese.dyndns.org>
In-Reply-To
<CAMP44s1h=MPT8vx6JrGjMZWJzRjndxYKoYgo+1Y_Mmv+gWXzaQ@mail.gmail.com>
Felipe Contreras <felipe.contreras@gmail.com> writes:
>> Here is my attempt to explain why none of them needs to be quoted:
>> ...
> I don't understand that.

That was my attempt to give the readers some context to understand what comes after that. If the original patch author does not understand it, perhaps it is an indication that the function is somewhat underdocumented.

>>  # wrapper: the name of an internal shell function that wraps the
> ...
> So in short; because the variables are used as function/command names.

Yes, and the fact that this is internal and we won't listen to clueless people who attempt "__git_complete 'foo bar' 'bar boz'" and complain are what make all of them safe to be used without any quotes around it.

Previous: Felipe ContrerasNext: Felipe Contreras
Message 7 of 11 in “completion: add new __git_complete helper”
  1. completion: add new __git_complete helperFelipe Contreras, May 14, 2012
  2. Felipe ContrerasMay 14, 2012
  3. Junio C HamanoMay 14, 2012
  4. Felipe ContrerasMay 14, 2012
  5. Junio C HamanoMay 14, 2012
  6. Felipe ContrerasMay 14, 2012
  7. Junio C HamanoMay 14, 2012
  8. Felipe ContrerasMay 14, 2012
  9. Junio C HamanoMay 15, 2012
  10. Junio C HamanoMay 15, 2012
  11. Felipe ContrerasMay 16, 2012

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.