Mercurial > vim
changeset 22083:c67a9d3b3683 v8.2.1591
patch 8.2.1591: using winheight('.') in tests works but is wrong
Commit: https://github.com/vim/vim/commit/c05d1c043a25c7a52dbf0a740d3f715a9d2691f7
Author: Bram Moolenaar <Bram@vim.org>
Date: Fri Sep 4 18:38:06 2020 +0200
patch 8.2.1591: using winheight('.') in tests works but is wrong
Problem: Using winheight('.') in tests works but is wrong.
Solution: Use winheight(0). (issue https://github.com/vim/vim/issues/6863)
| author | Bram Moolenaar <Bram@vim.org> |
|---|---|
| date | Fri, 04 Sep 2020 18:45:05 +0200 |
| parents | a7650f20d72e |
| children | d51d59ea5d56 |
| files | src/testdir/test_functions.vim src/testdir/test_quickfix.vim src/version.c |
| diffstat | 3 files changed, 8 insertions(+), 6 deletions(-) [+] |
line wrap: on
line diff
--- a/src/testdir/test_functions.vim +++ b/src/testdir/test_functions.vim @@ -1438,19 +1438,19 @@ func Test_setbufvar_options() let prev_id = win_getid() wincmd j - let wh = winheight('.') + let wh = winheight(0) let dummy_buf = bufnr('dummy_buf1', v:true) call setbufvar(dummy_buf, '&buftype', 'nofile') execute 'belowright vertical split #' . dummy_buf - call assert_equal(wh, winheight('.')) + call assert_equal(wh, winheight(0)) let dum1_id = win_getid() wincmd h - let wh = winheight('.') + let wh = winheight(0) let dummy_buf = bufnr('dummy_buf2', v:true) eval 'nofile'->setbufvar(dummy_buf, '&buftype') execute 'belowright vertical split #' . dummy_buf - call assert_equal(wh, winheight('.')) + call assert_equal(wh, winheight(0)) bwipe! call win_gotoid(prev_id)
--- a/src/testdir/test_quickfix.vim +++ b/src/testdir/test_quickfix.vim @@ -261,13 +261,13 @@ func XwindowTests(cchar) " Open the window Xopen 5 call assert_true(winnr('$') == 2 && getline('.') ==# '|| non-error 1' - \ && winheight('.') == 5) + \ && winheight(0) == 5) " Opening the window again, should move the cursor to that window wincmd t Xopen 7 call assert_true(winnr('$') == 2 && winnr() == 2 && - \ winheight('.') == 7 && + \ winheight(0) == 7 && \ getline('.') ==# '|| non-error 1') " :cnext in quickfix window should move to the next entry