Re: [PATCH v4 0/5] submodule: parallelize diff

From: Glen Choo <chooglen@google.com>
Date: Nov 23, 2022, 17:49 UTC
Message-ID: <kl6l35a9ef3c.fsf@chooglen-macbookpro.roam.corp.google.com>
In-Reply-To: <20221108184200.2813458-1-calvinwan@google.com>

Calvin Wan <calvinwan@google.com> writes:

Show 15 quoted lines

> Original cover letter for context:
> https://lore.kernel.org/git/20221011232604.839941-1-calvinwan@google.com/
>
> Changes since v3
>
> Renamed pipe_output_fn to duplicate_output_fn and now passes an
> additional "strbuf* out" parameter. Output is directly duplicated
> to that function rather than held in a separate variable.
> Slightly rewrote the tests to more accurately capture the expected
> output of duplicate_output_fn.
>
> Removed a patch that added an option to hide child process output.
> Child process output is now reset in status_duplicate_output.
>
> More style changes as suggested by Avar

For ease of navigation, here are the previous versions:

v3: https://lore.kernel.org/git/20221020232532.1128326-1-calvinwan@google.com/ v2: https://lore.kernel.org/git/20221011232604.839941-1-calvinwan@google.com/ v1: https://lore.kernel.org/git/20220922232947.631309-1-calvinwan@google.com/