Re: [PATCH 07/14] range-diff: respect diff_option.file rather than assuming 'stdout'

From: Eric Sunshine <sunshine@sunshineco.com>
Date: Jul 23, 2018, 23:20 UTC
Message-ID: <CAPig+cQtzs5ENzOR2sJVkZu13OF7iMnEcf4GG1jKGrzRhE0AUg@mail.gmail.com>
In-Reply-To: <CAGZ79kZ0OMrU_O5_rPdG7bNHRWQxVV1RguEg3g-pOa+kkgPCaA@mail.gmail.com>

On Mon, Jul 23, 2018 at 6:59 PM Stefan Beller <sbeller@google.com> wrote:

Show 10 quoted lines

> On Sun, Jul 22, 2018 at 2:58 AM Eric Sunshine <sunshine@sunshineco.com> wrote:
> > The actual diffs output by range-diff respect diff_option.file, which
> > range-diff passes down the call-chain, thus are destination-agnostic.
> > However, output_pair_header() is hard-coded to emit to 'stdout'. Fix
> > this by making output_pair_header() respect diff_option.file, as well.
> >
> > Signed-off-by: Eric Sunshine <sunshine@sunshineco.com>
>
> Depending how much the range diff series has progressed
> already, it might make sense to squash it there?

It could be done that way, though it would be nice for Dscho's range-diff series to land without another re-roll, and it looks like I'll probably be re-rolling this series, anyhow, to move show_interdiff() to diff-lib.c and to fix its signature. Junio could manage it as a "squash", but that's probably more work for him than for me to retain it in this series. *And*, this change _is_ required for this series, whereas it doesn't really matter for Dscho's series, even though it's an obvious "fix".

Anyhow, whichever way Junio and Dscho would like to play it is fine with me.