From a5de6770c5d7f331d13b34ece8dc3738b5a0af42 Mon Sep 17 00:00:00 2001 From: "Justin M. Keyes" Date: Sat, 19 May 2018 12:12:20 +0200 Subject: vim-patch:8.0.0304: assign test fails in the GUI Problem: Assign test fails in the GUI. Solution: Skip the test for setting t_k1. https://github.com/vim/vim/commit/1fb0d49803b4f57025ed6a43b1f5b9eb91524645 vim-patch:8.0.0354 --- src/nvim/testdir/test_assign.vim | 26 +++++++++++++++++++++++++- 1 file changed, 25 insertions(+), 1 deletion(-) (limited to 'src') diff --git a/src/nvim/testdir/test_assign.vim b/src/nvim/testdir/test_assign.vim index 3d2e7a8998..a75e89d5dd 100644 --- a/src/nvim/testdir/test_assign.vim +++ b/src/nvim/testdir/test_assign.vim @@ -3,7 +3,31 @@ func Test_no_type_checking() let v = 1 let v = [1,2,3] - let v = {'a':1, 'b':2} + let v = {'a': 1, 'b': 2} let v = 3.4 let v = 'hello' endfunc + +func Test_let_termcap() + " Nvim does not support `:set termcap`. + return + " Terminal code + let old_t_te = &t_te + let &t_te = "\[yes;" + call assert_match('t_te.*^[[yes;', execute("set termcap")) + let &t_te = old_t_te + + if exists("+t_k1") + " Key code + let old_t_k1 = &t_k1 + let &t_k1 = "that" + call assert_match('t_k1.*that', execute("set termcap")) + let &t_k1 = old_t_k1 + endif + + call assert_fails('let x = &t_xx', 'E15') + let &t_xx = "yes" + call assert_equal("yes", &t_xx) + let &t_xx = "" + call assert_fails('let x = &t_xx', 'E15') +endfunc -- cgit From 39f16a0fee3c13f587f80db51346d8e3785783ce Mon Sep 17 00:00:00 2001 From: "Justin M. Keyes" Date: Sat, 19 May 2018 12:24:54 +0200 Subject: vim-patch:8.0.0342: double free with EXITFREE and setting 'ttytype' Problem: Double free when compiled with EXITFREE and setting 'ttytype'. Solution: Avoid setting P_ALLOCED on 'ttytype'. (Dominique Pelle, closes vim/vim#1461) https://github.com/vim/vim/commit/673911457d6745b6b779eb769c2f41965592d12c --- src/nvim/option.c | 23 ++++++++++------------- src/nvim/testdir/test_options.vim | 20 ++++++++++++++++++++ 2 files changed, 30 insertions(+), 13 deletions(-) (limited to 'src') diff --git a/src/nvim/option.c b/src/nvim/option.c index 48c874196d..88c458b597 100644 --- a/src/nvim/option.c +++ b/src/nvim/option.c @@ -913,27 +913,24 @@ void set_number_default(char *name, long val) } #if defined(EXITFREE) -/* - * Free all options. - */ +/// Free all options. void free_all_options(void) { - int i; - - for (i = 0; options[i].fullname; i++) { + for (int i = 0; options[i].fullname; i++) { if (options[i].indir == PV_NONE) { - /* global option: free value and default value. */ - if (options[i].flags & P_ALLOCED && options[i].var != NULL) + // global option: free value and default value. + if ((options[i].flags & P_ALLOCED) && options[i].var != NULL) { free_string_option(*(char_u **)options[i].var); - if (options[i].flags & P_DEF_ALLOCED) + } + if (options[i].flags & P_DEF_ALLOCED) { free_string_option(options[i].def_val[VI_DEFAULT]); - } else if (options[i].var != VAR_WIN - && (options[i].flags & P_STRING)) - /* buffer-local option: free global value */ + } + } else if (options[i].var != VAR_WIN && (options[i].flags & P_STRING)) { + // buffer-local option: free global value free_string_option(*(char_u **)options[i].var); + } } } - #endif diff --git a/src/nvim/testdir/test_options.vim b/src/nvim/testdir/test_options.vim index f8c3161b40..738c703b7f 100644 --- a/src/nvim/testdir/test_options.vim +++ b/src/nvim/testdir/test_options.vim @@ -227,6 +227,26 @@ func Test_set_errors() call assert_fails('set t_foo=', 'E846:') endfunc +func Test_set_ttytype() + " Nvim does not support 'ttytype'. + if !has('nvim') && !has('gui_running') && has('unix') + " Setting 'ttytype' used to cause a double-free when exiting vim and + " when vim is compiled with -DEXITFREE. + set ttytype=ansi + call assert_equal('ansi', &ttytype) + call assert_equal(&ttytype, &term) + set ttytype=xterm + call assert_equal('xterm', &ttytype) + call assert_equal(&ttytype, &term) + " FIXME: "set ttytype=" gives E522 instead of E529 + " in travis on some builds. Why? Commented out this test for now. + " call assert_fails('set ttytype=', 'E529:') + call assert_fails('set ttytype=xxx', 'E522:') + set ttytype& + call assert_equal(&ttytype, &term) + endif +endfunc + func Test_complete() " Trailing single backslash used to cause invalid memory access. set complete=s\ -- cgit From fdd59ef901ad5439fc5f0be4125c67b31d131447 Mon Sep 17 00:00:00 2001 From: "Justin M. Keyes" Date: Sat, 19 May 2018 12:30:49 +0200 Subject: vim-patch:8.0.0559: setting ttytype to xxx does not always fail Problem: Setting ttytype to xxx does not always fail as expected. (Marvin Schmidt) Solution: Catch both possible errors. (closes vim/vim#1601) https://github.com/vim/vim/commit/f803a7697808f63949ea26fc256d4c9a8647089e --- src/nvim/testdir/test_options.vim | 20 ++++++++++++++++---- 1 file changed, 16 insertions(+), 4 deletions(-) (limited to 'src') diff --git a/src/nvim/testdir/test_options.vim b/src/nvim/testdir/test_options.vim index 738c703b7f..eb42e35bd3 100644 --- a/src/nvim/testdir/test_options.vim +++ b/src/nvim/testdir/test_options.vim @@ -238,10 +238,22 @@ func Test_set_ttytype() set ttytype=xterm call assert_equal('xterm', &ttytype) call assert_equal(&ttytype, &term) - " FIXME: "set ttytype=" gives E522 instead of E529 - " in travis on some builds. Why? Commented out this test for now. - " call assert_fails('set ttytype=', 'E529:') - call assert_fails('set ttytype=xxx', 'E522:') + " "set ttytype=" gives E522 instead of E529 + " in travis on some builds. Why? Catch both for now + try + set ttytype= + call assert_report('set ttype= did not fail') + catch /E529\|E522/ + endtry + + " Some systems accept any terminal name and return dumb settings, + " check for failure of finding the entry and for missing 'cm' entry. + try + set ttytype=xxx + call assert_report('set ttype=xxx did not fail') + catch /E522\|E437/ + endtry + set ttytype& call assert_equal(&ttytype, &term) endif -- cgit