diff options
| author | Johannes Schindelin <johannes.schindelin@gmx.de> | 2016-06-20 12:56:01 +0200 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2016-06-20 12:50:39 -0700 |
| commit | ef74594e42e57308ef67b4bef03e42c137e7ad4b (patch) | |
| tree | 22283af886e39f6261031e28495feb92bfa8b56b | |
| parent | f95d337dd455076a9a8e0638db3430ad09d24937 (diff) | |
| download | git-ef74594e42e57308ef67b4bef03e42c137e7ad4b.tar.gz | |
format-patch: use stdout directly
Earlier, we freopen()ed stdout in order to write patches to files.
That forced us to duplicate stdout (naming it "realstdout") because we
*still* wanted to be able to report the file names.
As we do not abuse stdout that way anymore, we no longer need to
duplicate stdout, either.
Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
| -rw-r--r-- | builtin/log.c | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/builtin/log.c b/builtin/log.c index db034a87a3..5a889d5442 100644 --- a/builtin/log.c +++ b/builtin/log.c @@ -796,7 +796,6 @@ static int git_format_config(const char *var, const char *value, void *cb) return git_log_config(var, value, cb); } -static FILE *realstdout = NULL; static const char *output_directory = NULL; static int outdir_offset; @@ -822,7 +821,7 @@ static int open_next_file(struct commit *commit, const char *subject, fmt_output_subject(&filename, subject, rev); if (!quiet) - fprintf(realstdout, "%s\n", filename.buf + outdir_offset); + printf("%s\n", filename.buf + outdir_offset); if ((rev->diffopt.file = fopen(filename.buf, "w")) == NULL) return error(_("Cannot open patch file %s"), filename.buf); @@ -1629,9 +1628,6 @@ int cmd_format_patch(int argc, const char **argv, const char *prefix) get_patch_ids(&rev, &ids); } - if (!use_stdout) - realstdout = xfdopen(xdup(1), "w"); - if (prepare_revision_walk(&rev)) die(_("revision walk setup failed")); rev.boundary = 1; |