Mercurial > vim
changeset 32062:8c9f63a518f5 v9.0.1362
patch 9.0.1362: ml_get error when going to another tab
Commit: https://github.com/vim/vim/commit/99ad3a8bb95c6f860545a050472b6181e33bac1a
Author: Bram Moolenaar <Bram@vim.org>
Date: Mon Feb 27 17:18:01 2023 +0000
patch 9.0.1362: ml_get error when going to another tab
Problem: ml_get error when going to another tab. (Daniel J. Perry)
Solution: Do not call update_topline() if "curwin" is invalid.
(closes #11907)
author | Bram Moolenaar <Bram@vim.org> |
---|---|
date | Mon, 27 Feb 2023 18:30:04 +0100 |
parents | b2412874362f |
children | b11f2c5c3978 |
files | src/testdir/test_tabpage.vim src/version.c src/window.c |
diffstat | 3 files changed, 22 insertions(+), 5 deletions(-) [+] |
line wrap: on
line diff
--- a/src/testdir/test_tabpage.vim +++ b/src/testdir/test_tabpage.vim @@ -872,4 +872,19 @@ func Test_tabpage_alloc_failure() call assert_equal(1, tabpagenr('$')) endfunc +" this was giving ml_get errors +func Test_tabpage_last_line() + enew + call setline(1, repeat(['a'], &lines + 5)) + $ + tabnew + call setline(1, repeat(['b'], &lines + 20)) + $ + tabNext + call assert_equal('a', getline('.')) + + bwipe! + bwipe! +endfunc + " vim: shiftwidth=2 sts=2 expandtab
--- a/src/version.c +++ b/src/version.c @@ -696,6 +696,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ /**/ + 1362, +/**/ 1361, /**/ 1360,
--- a/src/window.c +++ b/src/window.c @@ -5273,15 +5273,15 @@ win_enter_ext(win_T *wp, int flags) int curwin_invalid = (flags & WEE_CURWIN_INVALID); int did_decrement = FALSE; - if (wp == curwin && !curwin_invalid) // nothing to do + if (wp == curwin && curwin_invalid == 0) // nothing to do return FALSE; #ifdef FEAT_JOB_CHANNEL - if (!curwin_invalid) + if (curwin_invalid == 0) leaving_window(curwin); #endif - if (!curwin_invalid && (flags & WEE_TRIGGER_LEAVE_AUTOCMDS)) + if (curwin_invalid == 0 && (flags & WEE_TRIGGER_LEAVE_AUTOCMDS)) { /* * Be careful: If autocommands delete the window, return now. @@ -5309,13 +5309,13 @@ win_enter_ext(win_T *wp, int flags) // Might need to scroll the old window before switching, e.g., when the // cursor was moved. - if (*p_spk == 'c') + if (*p_spk == 'c' && curwin_invalid == 0) update_topline(); // may have to copy the buffer options when 'cpo' contains 'S' if (wp->w_buffer != curbuf) buf_copy_options(wp->w_buffer, BCO_ENTER | BCO_NOHELP); - if (!curwin_invalid) + if (curwin_invalid == 0) { prevwin = curwin; // remember for CTRL-W p curwin->w_redr_status = TRUE;