diff options
author | Jeff King <peff@peff.net> | 2009-12-30 05:56:16 -0500 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2010-01-05 23:41:51 -0800 |
commit | bac8037081735a49bccdc3b3b1457129f9bcc451 (patch) | |
tree | 09294e39702ca4a901e93bd23703610a047a3e55 /editor.c | |
parent | f445644fd28d31a828731a618e9a9c79be89efd2 (diff) | |
download | git-bac8037081735a49bccdc3b3b1457129f9bcc451.tar.gz |
editor: use run_command's shell feature
Now that run_command implements the same code in a more
general form, we can make use of it.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'editor.c')
-rw-r--r-- | editor.c | 21 |
1 files changed, 2 insertions, 19 deletions
@@ -36,26 +36,9 @@ int launch_editor(const char *path, struct strbuf *buffer, const char *const *en return error("Terminal is dumb, but EDITOR unset"); if (strcmp(editor, ":")) { - size_t len = strlen(editor); - int i = 0; - int failed; - const char *args[6]; - struct strbuf arg0 = STRBUF_INIT; + const char *args[] = { editor, path, NULL }; - if (strcspn(editor, "|&;<>()$`\\\"' \t\n*?[#~=%") != len) { - /* there are specials */ - strbuf_addf(&arg0, "%s \"$@\"", editor); - args[i++] = "sh"; - args[i++] = "-c"; - args[i++] = arg0.buf; - } - args[i++] = editor; - args[i++] = path; - args[i] = NULL; - - failed = run_command_v_opt_cd_env(args, 0, NULL, env); - strbuf_release(&arg0); - if (failed) + if (run_command_v_opt_cd_env(args, RUN_USING_SHELL, NULL, env)) return error("There was a problem with the editor '%s'.", editor); } |