Re: [PATCH i18n 01/11] Add three convenient format printing functions with \n automatically appended

From: Jonathan Nieder <jrnieder@gmail.com>
Date: Apr 16, 2012, 17:26 UTC
Message-ID: <20120416172654.GU5813@burratino>
In-Reply-To: <1334580603-11577-2-git-send-email-pclouds@gmail.com>

Nguyễn Thái Ngọc Duy wrote:

Show 5 quoted lines

>     printf("hello world\n");
>
> can be converted to
>
>     printf_ln(_("hello world"));

Fun.

[...]

> --- a/strbuf.c
> +++ b/strbuf.c
> @@ -464,3 +464,36 @@ void strbuf_addstr_urlencode(struct strbuf *sb, const char *s,

[...]

Show 9 quoted lines

> +int printf_ln(const char *fmt, ...)
> +{
> +	int ret;
> +	va_list ap;
> +	va_start(ap, fmt);
> +	ret = vprintf(fmt, ap);
> +	va_end(ap);
> +	if (ret >= 0)
> +		ret += printf("\n");

What happens if the second printf fails?

$ 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.