summaryrefslogtreecommitdiffstats
path: root/add-patch.c
diff options
context:
space:
mode:
authorRubén Justo <rjusto@gmail.com>2024-04-29 20:37:12 +0200
committerJunio C Hamano <gitster@pobox.com>2024-04-30 21:02:50 +0200
commit26998ed2a2ed2e36103da7131e44cd6a0c8d4565 (patch)
tree2eb6d3cd287a6cc725d5df6f0a0215ef3a3be797 /add-patch.c
parentadd-patch: do not show UI messages on stderr (diff)
downloadgit-26998ed2a2ed2e36103da7131e44cd6a0c8d4565.tar.xz
git-26998ed2a2ed2e36103da7131e44cd6a0c8d4565.zip
add-patch: response to unknown command
When the user gives an unknown command to the "add -p" prompt, the list of accepted commands with their explanation is given. This is the same output they get when they say '?'. However, the unknown command may be due to a user input error rather than the user not knowing the valid command. To reduce the likelihood of user confusion and error repetition, instead of displaying the list of accepted commands, display a short error message with the unknown command received, as feedback to the user. Include a reminder about the current command '?' in the new message, to guide the user if they want help. Signed-off-by: Rubén Justo <rjusto@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'add-patch.c')
-rw-r--r--add-patch.c5
1 files changed, 4 insertions, 1 deletions
diff --git a/add-patch.c b/add-patch.c
index 43abd57ef6..85c4770572 100644
--- a/add-patch.c
+++ b/add-patch.c
@@ -1666,7 +1666,7 @@ soft_increment:
}
} else if (s->answer.buf[0] == 'p') {
rendered_hunk_index = -1;
- } else {
+ } else if (s->answer.buf[0] == '?') {
const char *p = _(help_patch_remainder), *eol = p;
color_fprintf(stdout, s->s.help_color, "%s",
@@ -1690,6 +1690,9 @@ soft_increment:
color_fprintf_ln(stdout, s->s.help_color,
"%.*s", (int)(eol - p), p);
}
+ } else {
+ err(s, _("Unknown command '%s' (use '?' for help)"),
+ s->answer.buf);
}
}