]> granicus.if.org Git - vim/commitdiff
patch 8.1.2195: Vim does not exit when the terminal window is last window v8.1.2195
authorBram Moolenaar <Bram@vim.org>
Sun, 20 Oct 2019 19:15:15 +0000 (21:15 +0200)
committerBram Moolenaar <Bram@vim.org>
Sun, 20 Oct 2019 19:15:15 +0000 (21:15 +0200)
Problem:    Vim does not exit when closing a terminal window and it is the
            last window.
Solution:   Exit Vim if the closed terminal window is the last one.
            (closes #4539)

runtime/doc/terminal.txt
src/ex_docmd.c
src/proto/ex_docmd.pro
src/terminal.c
src/testdir/test_terminal.vim
src/version.c

index fc5031dd46a1daf69be1f27a8a12a75c38a9318c..6bf04eb8300532a70b06b74df0d784a5fd579839 100644 (file)
@@ -193,6 +193,7 @@ Command syntax ~
                        Supported [options] are:
                        ++close         The terminal window will close
                                        automatically when the job terminates.
+                                       |terminal-close|
                        ++noclose       The terminal window will NOT close
                                        automatically when the job terminates.
                        ++open          When the job terminates and no window
@@ -267,6 +268,12 @@ hidden, the job keeps running.  The `:buffer` command can be used to turn the
 current window into a terminal window.  If there are unsaved changes this
 fails, use ! to force, as usual.
 
+                                                       *terminal-close*
+When the terminal window is closed, e.g. when the shell exits and "++close"
+argument was used, and this is the last normal Vim window, then Vim will exit.
+This is like using |:quit| in a normal window. Help and preview windows are
+not counted.
+
 To have a background job run without a window, and open the window when it's
 done, use options like this: >
        :term ++hidden ++open make
index 1d0476946d9b7c31fe557277936c2aa131403687..d9fe7de75e1bddc6537da998719c6d50241e7e62 100644 (file)
@@ -87,7 +87,6 @@ static char_u *replace_makeprg(exarg_T *eap, char_u *p, char_u **cmdlinep);
 static char_u  *repl_cmdline(exarg_T *eap, char_u *src, int srclen, char_u *repl, char_u **cmdlinep);
 static void    ex_highlight(exarg_T *eap);
 static void    ex_colorscheme(exarg_T *eap);
-static void    ex_quit(exarg_T *eap);
 static void    ex_cquit(exarg_T *eap);
 static void    ex_quit_all(exarg_T *eap);
 static void    ex_close(exarg_T *eap);
@@ -4842,8 +4841,9 @@ before_quit_autocmds(win_T *wp, int quit_all, int forceit)
 /*
  * ":quit": quit current window, quit Vim if the last window is closed.
  * ":{nr}quit": quit window {nr}
+ * Also used when closing a terminal window that's the last one.
  */
-    static void
+    void
 ex_quit(exarg_T *eap)
 {
     win_T      *wp;
index cc64ac882081c295a7d6bcdcafe50babd33c39bc..f5942f5c1a97597ad37293f324e6d0bf711729b4 100644 (file)
@@ -15,13 +15,14 @@ char_u *skip_range(char_u *cmd, int *ctx);
 void ex_ni(exarg_T *eap);
 int expand_filename(exarg_T *eap, char_u **cmdlinep, char **errormsgp);
 void separate_nextcmd(exarg_T *eap);
-char_u *skip_cmd_arg( char_u *p, int rembs);
+char_u *skip_cmd_arg(char_u *p, int rembs);
 int get_bad_opt(char_u *p, exarg_T *eap);
 int ends_excmd(int c);
 char_u *find_nextcmd(char_u *p);
 char_u *check_nextcmd(char_u *p);
 char_u *get_command_name(expand_T *xp, int idx);
 void not_exiting(void);
+void ex_quit(exarg_T *eap);
 void tabpage_close(int forceit);
 void tabpage_close_other(tabpage_T *tp, int forceit);
 void handle_drop(int filec, char_u **filev, int split, void (*callback)(void *), void *cookie);
@@ -30,8 +31,8 @@ void ex_splitview(exarg_T *eap);
 void tabpage_new(void);
 void do_exedit(exarg_T *eap, win_T *old_curwin);
 void free_cd_dir(void);
-void post_chdir(cdscope_T cdscope);
-int changedir_func(char_u *new_dir, int forceit, cdscope_T cdscope);
+void post_chdir(cdscope_T scope);
+int changedir_func(char_u *new_dir, int forceit, cdscope_T scope);
 void ex_cd(exarg_T *eap);
 void do_sleep(long msec);
 void ex_may_print(exarg_T *eap);
@@ -47,8 +48,6 @@ void exec_normal(int was_typed, int use_vpeekc, int may_use_terminal_loop);
 int find_cmdline_var(char_u *src, int *usedlen);
 char_u *eval_vars(char_u *src, char_u *srcstart, int *usedlen, linenr_T *lnump, char **errormsg, int *escaped);
 char_u *expand_sfile(char_u *arg);
-int put_eol(FILE *fd);
-int put_line(FILE *fd, char *s);
 void dialog_msg(char_u *buff, char *format, char_u *fname);
 void set_no_hlsearch(int flag);
 int is_loclist_cmd(int cmdidx);
index 7b2d43bf256c568dc7423c11f9a073003b9c6d9b..07b2e973e076ef94fc5d9abb6ecc70dc24856347 100644 (file)
@@ -3073,6 +3073,16 @@ term_after_channel_closed(term_T *term)
            aco_save_T  aco;
            int         do_set_w_closing = term->tl_buffer->b_nwindows == 0;
 
+           // If this is the last normal window: exit Vim.
+           if (term->tl_buffer->b_nwindows > 0 && only_one_window())
+           {
+               exarg_T ea;
+
+               vim_memset(&ea, 0, sizeof(ea));
+               ex_quit(&ea);
+               return TRUE;
+           }
+
            // ++close or term_finish == "close"
            ch_log(NULL, "terminal job finished, closing window");
            aucmd_prepbuf(&aco, term->tl_buffer);
index c4d3e528c668480054e916d192903a79cde5136f..f506a39f68d9b414f3c9987cff568b261bea2dae 100644 (file)
@@ -1051,6 +1051,32 @@ func Test_terminal_qall_prompt()
   quit
 endfunc
 
+" Run Vim in a terminal, then start a terminal window with a shell and check
+" that Vim exits if it is closed.
+func Test_terminal_exit()
+  CheckRunVimInTerminal
+
+  let lines =<< trim END
+     let winid = win_getid()
+     help
+     term
+     let termid = win_getid()
+     call win_gotoid(winid)
+     close
+     call win_gotoid(termid)
+  END
+  call writefile(lines, 'XtermExit')
+  let buf = RunVimInTerminal('-S XtermExit', #{rows: 10})
+  let job = term_getjob(buf)
+  call WaitForAssert({-> assert_equal("run", job_status(job))})
+
+  " quit the shell, it will make Vim exit
+  call term_sendkeys(buf, "exit\<CR>")
+  call WaitForAssert({-> assert_equal("dead", job_status(job))})
+
+  call delete('XtermExit')
+endfunc
+
 func Test_terminal_open_autocmd()
   augroup repro
     au!
index 38696fa45334477ed40a09757cee91fb4616f4d8..c378c1bba7c22c486117f87f817dc4c5b8937551 100644 (file)
@@ -741,6 +741,8 @@ static char *(features[]) =
 
 static int included_patches[] =
 {   /* Add new patch number below this line */
+/**/
+    2195,
 /**/
     2194,
 /**/