Re: [PATCH v4 03/12] pkt-line: add packet_flush_gentle()

From: Junio C Hamano <gitster@pobox.com>
Date: Aug 4, 2016, 16:16 UTC
Message-ID: <xmqqa8gs5x30.fsf@gitster.mtv.corp.google.com>
In-Reply-To: <20160803213920.jg3eshy57bsldqjh@sigill.intra.peff.net>

Jeff King <peff@peff.net> writes:

Show 11 quoted lines

>   2. It calls check_pipe(), which will turn EPIPE into death-by-SIGPIPE
>      (in case you had for some reason ignored SIGPIPE).
> ...
>
> Thinking about (2), I'd go so far as to say that the trace actually
> should just be using:
>
>   if (write_in_full(...) < 0)
> 	warning("unable to write trace to ...: %s", strerror(errno));
>
> and we should get rid of write_or_whine_pipe entirely.

I like the simplicity the above suggestion gives us.

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