Subject: Re: [PATCH v7 10/12] run-command: poll child stdin in addition to stdout

From: Junio C Hamano <[email-masked]>
Date: Mon, 26 Jan 2026 16:11:03 -0800

Emily Shaffer <[email-masked]> writes:

>> +       /* for each potential child slot, prepare two pollfd entries */
>> +       for (size_t i = 0; i < opts->processes; i++) {
>> +               if (child_is_working(&pp->children[i]) &&
>> +                   pp->children[i].process.err > 0) {
>
> I only had the one tiny nit on this patch, which was to wonder if
> checking for pp->children[i].process.err is something that should also
> be behind a conveniently-named helper like child_is_working().

We already have

 - child_is_ready_for_cleanup()
 - child_is_receiving_input()
 - child_is_working()

What should the "child is working and process.err is positive" be
called?  child_is_spewing_error()?

Thanks.