summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMatthieu Moy <Matthieu.Moy@imag.fr>2013-07-16 10:05:36 +0200
committerJunio C Hamano <gitster@pobox.com>2013-07-17 17:50:56 -0700
commitd09cd15d19de23aca532a85f6d27a71b2baceb3f (patch)
tree83ba6d8aa00c4b0d7ae10500fa04cfaf1ee4d155
parent8ade9b140f099540bcdec5bcf660faaf05b5e3ee (diff)
downloadgit-d09cd15d19de23aca532a85f6d27a71b2baceb3f.tar.gz
diff: allow --no-patch as synonym for -s
This follows the usual convention of having a --no-foo option to negate --foo. Signed-off-by: Matthieu Moy <Matthieu.Moy@imag.fr> Reviewed-by: Jonathan Nieder <jrnieder@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>
-rw-r--r--Documentation/rev-list-options.txt1
-rw-r--r--diff.c2
-rwxr-xr-xt/t4000-diff-format.sh12
3 files changed, 14 insertions, 1 deletions
diff --git a/Documentation/rev-list-options.txt b/Documentation/rev-list-options.txt
index 3bdbf5e856..1acef370d1 100644
--- a/Documentation/rev-list-options.txt
+++ b/Documentation/rev-list-options.txt
@@ -839,5 +839,6 @@ options may be given. See linkgit:git-diff-files[1] for more options.
Show the tree objects in the diff output. This implies '-r'.
-s::
+--no-patch::
Suppress diff output.
endif::git-rev-list[]
diff --git a/diff.c b/diff.c
index f0b3e7cfe3..cc41d88c98 100644
--- a/diff.c
+++ b/diff.c
@@ -3540,7 +3540,7 @@ int diff_opt_parse(struct diff_options *options, const char **av, int ac)
options->output_format |= DIFF_FORMAT_NAME;
else if (!strcmp(arg, "--name-status"))
options->output_format |= DIFF_FORMAT_NAME_STATUS;
- else if (!strcmp(arg, "-s"))
+ else if (!strcmp(arg, "-s") || !strcmp(arg, "--no-patch"))
options->output_format |= DIFF_FORMAT_NO_OUTPUT;
else if (!prefixcmp(arg, "--stat"))
/* --stat, --stat-width, --stat-name-width, or --stat-count */
diff --git a/t/t4000-diff-format.sh b/t/t4000-diff-format.sh
index 2b5dffc3c4..3b9a9ae682 100755
--- a/t/t4000-diff-format.sh
+++ b/t/t4000-diff-format.sh
@@ -59,4 +59,16 @@ test_expect_success 'validate git diff-files -p output.' '
compare_diff_patch expected actual
'
+test_expect_success 'git diff-files -s after editing work tree' '
+ git diff-files -s >actual 2>err &&
+ test_must_be_empty actual &&
+ test_must_be_empty err
+'
+
+test_expect_success 'git diff-files --no-patch as synonym for -s' '
+ git diff-files --no-patch >actual 2>err &&
+ test_must_be_empty actual &&
+ test_must_be_empty err
+'
+
test_done