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

Re: [PATCH 1/4] Add a simple option parser for use by builtin-commit.c.

From
Johannes Schindelin <johannes.schindelin@gmx.de>
Date
Oct 1, 2007, 10:14 UTC
Message-ID
<Pine.LNX.4.64.0710011114310.28395@racer.site>
In-Reply-To
<20070930131133.GA11209@diku.dk>
Hi,
On Sun, 30 Sep 2007, Jonas Fonseca wrote:
> Also, I think for this to be more usable for other built-in programs it 
> shouldn't modify argv, but instead take both argc and argv (so we don't 
> need to have code like "*++(*argv)" ;), parse _all_ options in one go, 
> and return the index (of argv) for any remaining options.

We _have_ to modify argv. For example, "git log master -p" is perfectly valid.

Ciao, Dscho

Previous: Jonas FonsecaNext: Jonas Fonseca
Message 10 of 17 in “Add a simple option parser for use by builtin-commit.c.”
  1. 1/4 Add a simple option parser for use by builtin-commit.c.Kristian Høgsberg, Sep 27, 2007
  2. 2/4 This exports the update() function from builtin-add.c asKristian Høgsberg, Sep 27, 2007
  3. 3/4 Implement git commit as a builtin command.Kristian Høgsberg, Sep 27, 2007
  4. 4/4 Move launch_editor() and stripspace() to new file editor.c.Kristian Høgsberg, Sep 27, 2007
  5. Junio C HamanoSep 27, 2007
  6. Kristian HøgsbergOct 3, 2007
  7. Johannes SchindelinSep 27, 2007
  8. Junio C HamanoSep 27, 2007
  9. Jonas FonsecaSep 30, 2007
  10. Johannes SchindelinOct 1, 2007
  11. Jonas FonsecaOct 1, 2007
  12. Johannes SchindelinOct 1, 2007
  13. Jeff KingOct 1, 2007
  14. Kristian HøgsbergOct 1, 2007
  15. Johannes SchindelinOct 1, 2007
  16. Jonas FonsecaOct 3, 2007
  17. Kristian HøgsbergOct 3, 2007

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.