range-diff: respect diff_option.file rather than assuming 'stdout'

sunshineco · gitster · commit 87f1b2d45ca7 · 2018-08-14T14:24:56.000-07:00
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 &lt;sunshine@sunshineco.com&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
diff --git a/range-diff.c b/range-diff.c
@@ -323,7 +323,7 @@ static void output_pair_header(struct diff_options *diffopt,
 	}
 	strbuf_addf(buf, "%s\n", color_reset);
 
-	fwrite(buf->buf, buf->len, 1, stdout);
+	fwrite(buf->buf, buf->len, 1, diffopt->file);
 }
 
 static struct userdiff_driver no_func_name = {

Original file line number	Diff line number	Diff line change
`@@ -323,7 +323,7 @@ static void output_pair_header(struct diff_options *diffopt,`
`323`	`323`	`}`
`324`	`324`	`strbuf_addf(buf, "%s\n", color_reset);`
`325`	`325`
`326`		`- fwrite(buf->buf, buf->len, 1, stdout);`
	`326`	`+ fwrite(buf->buf, buf->len, 1, diffopt->file);`
`327`	`327`	`}`
`328`	`328`
`329`	`329`	`static struct userdiff_driver no_func_name = {`