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

Re: [PATCHv2 1/2] parse-options: add PARSE_OPT_LITERAL_ARGHELP for complicated argh's

From
Stephen Boyd <bebarino@gmail.com>
Date
May 21, 2009, 19:03 UTC
Message-ID
<780e0a6b0905211203g3223fe3fp5eed27ebf244394a@mail.gmail.com>
In-Reply-To
<4A1586A7.9000909@lsrfire.ath.cx>

On Thu, May 21, 2009 at 9:51 AM, René Scharfe <rene.scharfe@lsrfire.ath.cx> wrote:

Show 15 quoted lines
> Junio C Hamano schrieb:
>> Hmmm, why does this break t0040 (I am queuing this on top of 5acb3e5)?
>
> Probably because it changes this:
>
>        pos += fprintf(...);
>
> into this (simplified, usage_argh() expanded):
>
>        pos += pos + fprintf(...);
>
> usage_argh() doesn't need the parameter pos.
>
> René
>

Woops. I thought I ran the tests but I guess I didn't. This is the correct fix, thanks.

Previous: René ScharfeNext: Stephen Boyd
Message 5 of 6 in “[PATCHv2 1/2] parse-options: add PARSE_OPT_LITERAL_ARGHELP for complicated argh's”
  1. Stephen BoydMay 21, 2009
  2. 2/2 show-branch: migrate to parse-options APIStephen Boyd, May 21, 2009
  3. Junio C HamanoMay 21, 2009
  4. René ScharfeMay 21, 2009
  5. Stephen BoydMay 21, 2009
  6. Stephen BoydMay 21, 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.