summaryrefslogtreecommitdiff
path: root/shell.c
diff options
context:
space:
mode:
authorGreg Brockman <gdb@MIT.EDU>2010-08-27 01:36:13 -0400
committerJunio C Hamano <gitster@pobox.com>2010-08-27 10:43:59 -0700
commit9f29fe9a779e55e7387577cd7f712de0b13b5cd6 (patch)
tree20554d1dd73ab8ebd851a01ed94ba3aa91edbecc /shell.c
parent1666246b702379275fe87cbbbf6111a74d00d825 (diff)
downloadgit-9f29fe9a779e55e7387577cd7f712de0b13b5cd6.tar.gz
shell: Display errors from improperly-formatted command lines
The interface for split_cmdline has changed such that the caller holds responsibility for printing any error messages. This patch changes the git shell to print these error messages as appropriate. Signed-off-by: Greg Brockman <gdb@mit.edu> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'shell.c')
-rw-r--r--shell.c16
1 files changed, 13 insertions, 3 deletions
diff --git a/shell.c b/shell.c
index f0f6c2d3be..dea4cfdd2c 100644
--- a/shell.c
+++ b/shell.c
@@ -73,8 +73,10 @@ static void run_shell(void)
const char *prog;
char *full_cmd;
char *rawargs;
+ char *split_args;
const char **argv;
int code;
+ int count;
fprintf(stderr, "git> ");
if (strbuf_getline(&line, stdin, '\n') == EOF) {
@@ -84,7 +86,12 @@ static void run_shell(void)
}
strbuf_trim(&line);
rawargs = strbuf_detach(&line, NULL);
- if (split_cmdline(rawargs, &argv) == -1) {
+ split_args = xstrdup(rawargs);
+ count = split_cmdline(split_args, &argv);
+ if (count < 0) {
+ fprintf(stderr, "invalid command format '%s': %s\n", rawargs,
+ split_cmdline_strerror(count));
+ free(split_args);
free(rawargs);
continue;
}
@@ -128,6 +135,7 @@ int main(int argc, char **argv)
const char **user_argv;
struct commands *cmd;
int devnull_fd;
+ int count;
/*
* Always open file descriptors 0/1/2 to avoid clobbering files
@@ -190,7 +198,8 @@ int main(int argc, char **argv)
}
cd_to_homedir();
- if (split_cmdline(prog, &user_argv) != -1) {
+ count = split_cmdline(prog, &user_argv);
+ if (count >= 0) {
if (is_valid_cmd_name(user_argv[0])) {
prog = make_cmd(user_argv[0]);
user_argv[0] = prog;
@@ -201,6 +210,7 @@ int main(int argc, char **argv)
die("unrecognized command '%s'", argv[2]);
} else {
free(prog);
- die("invalid command format '%s'", argv[2]);
+ die("invalid command format '%s': %s", argv[2],
+ split_cmdline_strerror(count));
}
}