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

Re: [PATCH] revert: add missing va_end

From
Junio C Hamano <gitster@pobox.com>
Date
Apr 22, 2012, 05:20 UTC
Message-ID
<xmqqd370mjgd.fsf@junio.mtv.corp.google.com>
In-Reply-To
<20120421211520.2797.87418.chriscool@tuxfamily.org>
Christian Couder <chriscool@tuxfamily.org> writes:
Show 14 quoted lines
> Signed-off-by: Christian Couder <chriscool@tuxfamily.org>
> ---
>  builtin/revert.c |    1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/builtin/revert.c b/builtin/revert.c
> index e6840f2..e4ed023 100644
> --- a/builtin/revert.c
> +++ b/builtin/revert.c
> @@ -86,6 +86,7 @@ static void verify_opt_mutually_compatible(const char *me, ...)
>  				break;
>  		}
>  	}
> +	va_end(ap);
Yeah, the original from v1.7.8-rc0~141^2~2 is obviously wrong.  Thanks.
Previous: Christian Couder
Message 2 of 2 in “revert: add missing va_end”
  1. revert: add missing va_endChristian Couder, Apr 21, 2012
  2. Junio C HamanoApr 22, 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.