From fbe0b24ca53111208b8ecd920cc112d0be13f93c Mon Sep 17 00:00:00 2001 From: Johannes Schindelin Date: Wed, 6 Dec 2006 16:45:42 +0100 Subject: [PATCH] merge-file: support -p and -q; fix compile warnings Now merge-file also understands --stdout and --quiet options. While at it, two compile warnings were fixed. Signed-off-by: Johannes Schindelin Signed-off-by: Junio C Hamano --- builtin-merge-file.c | 25 ++++++++++++++++--------- 1 file changed, 16 insertions(+), 9 deletions(-) diff --git a/builtin-merge-file.c b/builtin-merge-file.c index 41fb92edf..6c4c3a351 100644 --- a/builtin-merge-file.c +++ b/builtin-merge-file.c @@ -2,7 +2,7 @@ #include "xdiff/xdiff.h" static const char merge_file_usage[] = -"git merge-file [-L name1 [-L orig [-L name2]]] file1 orig_file file2"; +"git merge-file [-p | --stdout] [-q | --quiet] [-L name1 [-L orig [-L name2]]] file1 orig_file file2"; static int read_file(mmfile_t *ptr, const char *filename) { @@ -27,16 +27,23 @@ int cmd_merge_file(int argc, char **argv, char **envp) mmfile_t mmfs[3]; mmbuffer_t result = {NULL, 0}; xpparam_t xpp = {XDF_NEED_MINIMAL}; - int ret = 0, i = 0; + int ret = 0, i = 0, to_stdout = 0; while (argc > 4) { - if (!strcmp(argv[1], "-L")) { + if (!strcmp(argv[1], "-L") && i < 3) { names[i++] = argv[2]; - argc -= 2; - argv += 2; - continue; - } - usage(merge_file_usage); + argc--; + argv++; + } else if (!strcmp(argv[1], "-p") || + !strcmp(argv[1], "--stdout")) + to_stdout = 1; + else if (!strcmp(argv[1], "-q") || + !strcmp(argv[1], "--quiet")) + freopen("/dev/null", "w", stderr); + else + usage(merge_file_usage); + argc--; + argv++; } if (argc != 4) @@ -57,7 +64,7 @@ int cmd_merge_file(int argc, char **argv, char **envp) if (ret >= 0) { char *filename = argv[1]; - FILE *f = fopen(filename, "wb"); + FILE *f = to_stdout ? stdout : fopen(filename, "wb"); if (!f) ret = error("Could not open %s for writing", filename); -- 2.26.2