Thanks for a prompt patch!
Marcin
-------- Original Message --------
On 4 Aug 2018, 15:54, Bram Moolenaar wrote:
> Patch 8.1.0238
> Problem: 'buftype' is cleared when using ":term ++hidden cat". (Marcin
> Szamotulski)
> Solution: Set the "options initialized" flag earlier. (closes #3278)
> Files: src/terminal.c, src/testdir/test_terminal.vim
>
> *** ../vim-8.1.0237/src/terminal.c 2018-07-08 20:49:03.427359658 +0200
> --- src/terminal.c 2018-08-04 16:52:59.228476711 +0200
> ***************
> *** 529,534 ****
> --- 529,536 ----
>
> set_string_option_direct((char_u *)"buftype", -1,
> (char_u *)"terminal", OPT_FREE|OPT_LOCAL, 0);
> + // Avoid that 'buftype' is reset when this buffer is entered.
> + curbuf->b_p_initialized = TRUE;
>
> /* Mark the buffer as not modifiable. It can only be made modifiable after
> * the job finished. */
> *** ../vim-8.1.0237/src/testdir/test_terminal.vim 2018-07-08
> 20:49:03.427359658 +0200
> --- src/testdir/test_terminal.vim 2018-08-04 16:51:08.821245793 +0200
> ***************
> *** 1605,1607 ****
> --- 1605,1623 ----
> set guioptions&
> call delete(filename)
> endfunc
> +
> + func Test_terminal_hidden()
> + if !has('unix')
> + return
> + endif
> + term ++hidden cat
> + let bnr = bufnr('$')
> + call assert_equal('terminal', getbufvar(bnr, '&buftype'))
> + exe 'sbuf ' . bnr
> + call assert_equal('terminal', &buftype)
> + call term_sendkeys(bnr, "asdf\<CR>")
> + call WaitForAssert({-> assert_match('asdf', term_getline(bnr, 2))})
> + call term_sendkeys(bnr, "\<C-D>")
> + call WaitForAssert({-> assert_equal('finished', term_getstatus(bnr))})
> + bwipe!
> + endfunc
> *** ../vim-8.1.0237/src/version.c 2018-08-04 15:13:31.034718130 +0200
> --- src/version.c 2018-08-04 16:53:18.336344327 +0200
> ***************
> *** 796,797 ****
> --- 796,799 ----
> { /* Add new patch number below this line */
> + /**/
> + 238,
> /**/
>
> --
> -rwxr-xr-x 1 root 24 Oct 29 1929 /bin/ed
> -rwxr-xr-t 4 root 131720 Jan 1 1970 /usr/ucb/vi
> -rwxr-xr-x 1 root 5.89824e37 Oct 22 1990 /usr/bin/emacs
>
> /// Bram Moolenaar -- [email protected] -- http://www.Moolenaar.net \\\
> /// sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
> \\\ an exciting new programming language -- http://www.Zimbu.org ///
> \\\ help me help AIDS victims -- http://ICCF-Holland.org ///
>
> --
> --
> You received this message from the "vim_dev" maillist.
> Do not top-post! Type your reply below the text you are replying to.
> For more information, visit http://www.vim.org/maillist.php
>
> ---
> You received this message because you are subscribed to the Google Groups
> "vim_dev" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to [email protected].
> For more options, visit https://groups.google.com/d/optout.
--
--
You received this message from the "vim_dev" maillist.
Do not top-post! Type your reply below the text you are replying to.
For more information, visit http://www.vim.org/maillist.php
---
You received this message because you are subscribed to the Google Groups
"vim_dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
