To: vim_dev@googlegroups.com Subject: Patch 8.2.1596 Fcc: outbox From: Bram Moolenaar Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit ------------ Patch 8.2.1596 Problem: Using win_screenpos('.') in tests works but is wrong. Solution: Use win_screenpos(0). Files: src/testdir/test_terminal3.vim *** ../vim-8.2.1595/src/testdir/test_terminal3.vim 2020-08-20 18:59:02.106779176 +0200 --- src/testdir/test_terminal3.vim 2020-09-05 15:04:36.179545770 +0200 *************** *** 551,557 **** let winpos = 50->getwinpos() call assert_equal(xroot, winpos[0]) call assert_equal(yroot, winpos[1]) ! let [winrow, wincol] = win_screenpos('.') let xoff = wincol * (has('gui_running') ? 14 : 7) + 100 let yoff = winrow * (has('gui_running') ? 20 : 10) + 200 call assert_inrange(xroot + 2, xroot + xoff, xpos) --- 551,557 ---- let winpos = 50->getwinpos() call assert_equal(xroot, winpos[0]) call assert_equal(yroot, winpos[1]) ! let [winrow, wincol] = win_screenpos(0) let xoff = wincol * (has('gui_running') ? 14 : 7) + 100 let yoff = winrow * (has('gui_running') ? 20 : 10) + 200 call assert_inrange(xroot + 2, xroot + xoff, xpos) *** ../vim-8.2.1595/src/version.c 2020-09-05 14:27:19.466565808 +0200 --- src/version.c 2020-09-05 15:05:02.675474628 +0200 *************** *** 756,757 **** --- 756,759 ---- { /* Add new patch number below this line */ + /**/ + 1596, /**/ -- Q: How many hardware engineers does it take to change a lightbulb? A: None. We'll fix it in software. /// Bram Moolenaar -- Bram@Moolenaar.net -- 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 ///