We have a hint shown when we are waiting for user's editor since
abfb04d0c7 (launch_editor(): indicate that Git waits for user input,
2017-12-07).
After showing the hint, we call start_command() which can return with an
error. Then we'll show "unable to start editor...", after having said
"Waiting for your editor...", which may be confusing.
Move the code to show the hint below the start_command().
Signed-off-by: Rubén Justo <rjusto@xxxxxxxxx>
---
editor.c | 28 ++++++++++++++--------------
1 file changed, 14 insertions(+), 14 deletions(-)
diff --git a/editor.c b/editor.c
index b67b802ddf..1da3a26f5d 100644
--- a/editor.c
+++ b/editor.c
@@ -64,9 +64,21 @@ static int launch_specified_editor(const char *editor, const char *path,
if (strcmp(editor, ":")) {
struct strbuf realpath = STRBUF_INIT;
struct child_process p = CHILD_PROCESS_INIT;
- int ret, sig;
- int print_waiting_for_editor = advice_enabled(ADVICE_WAITING_FOR_EDITOR) && isatty(2);
+ int ret, sig, print_waiting_for_editor;
+ strbuf_realpath(&realpath, path, 1);
+
+ strvec_pushl(&p.args, editor, realpath.buf, NULL);
+ if (env)
+ strvec_pushv(&p.env, (const char **)env);
+ p.use_shell = 1;
+ p.trace2_child_class = "editor";
+ if (start_command(&p) < 0) {
+ strbuf_release(&realpath);
+ return error("unable to start editor '%s'", editor);
+ }
+
+ print_waiting_for_editor = advice_enabled(ADVICE_WAITING_FOR_EDITOR) && isatty(2);
if (print_waiting_for_editor) {
/*
* A dumb terminal cannot erase the line later on. Add a
@@ -83,18 +95,6 @@ static int launch_specified_editor(const char *editor, const char *path,
fflush(stderr);
}
- strbuf_realpath(&realpath, path, 1);
-
- strvec_pushl(&p.args, editor, realpath.buf, NULL);
- if (env)
- strvec_pushv(&p.env, (const char **)env);
- p.use_shell = 1;
- p.trace2_child_class = "editor";
- if (start_command(&p) < 0) {
- strbuf_release(&realpath);
- return error("unable to start editor '%s'", editor);
- }
-
sigchain_push(SIGINT, SIG_IGN);
sigchain_push(SIGQUIT, SIG_IGN);
ret = finish_command(&p);
--
2.44.0.771.gbd07cf668b
