changeset 18483:54e5fa1f9cfc v8.1.2236

patch 8.1.2236: ml_get error if pattern matches beyond last line Commit: https://github.com/vim/vim/commit/bb26596242fa7db477e2cd706dd99f9a426b5f71 Author: Bram Moolenaar <Bram@vim.org> Date: Thu Oct 31 04:38:36 2019 +0100 patch 8.1.2236: ml_get error if pattern matches beyond last line Problem: Ml_get error if pattern matches beyond last line. Solution: Adjust position if needed. (Christian Brabandt, closes https://github.com/vim/vim/issues/5139)
author Bram Moolenaar <Bram@vim.org>
date Thu, 31 Oct 2019 04:45:03 +0100
parents f8975b6004e6
children 3e50e573ca25
files src/ex_cmds.c src/testdir/test_substitute.vim src/version.c
diffstat 3 files changed, 17 insertions(+), 0 deletions(-) [+]
line wrap: on
line diff
--- a/src/ex_cmds.c
+++ b/src/ex_cmds.c
@@ -3947,6 +3947,11 @@ do_sub(exarg_T *eap)
 		    VIM_CLEAR(sub_firstline);
 		}
 
+		// Match might be after the last line for "\n\zs" matching at
+		// the end of the last line.
+		if (lnum > curbuf->b_ml.ml_line_count)
+		    break;
+
 		if (sub_firstline == NULL)
 		{
 		    sub_firstline = vim_strsave(ml_get(sub_firstlnum));
--- a/src/testdir/test_substitute.vim
+++ b/src/testdir/test_substitute.vim
@@ -413,6 +413,7 @@ func Run_SubCmd_Tests(tests)
   for t in a:tests
     let start = line('.') + 1
     let end = start + len(t[2]) - 1
+    " TODO: why is there a one second delay the first time we get here?
     exe "normal o" . t[0]
     call cursor(start, 1)
     exe t[1]
@@ -716,3 +717,12 @@ one two
 
   close!
 endfunc
+
+func Test_sub_beyond_end()
+  new
+  call setline(1, '#')
+  let @/ = '^#\n\zs'
+  s///e
+  call assert_equal('#', getline(1))
+  bwipe!
+endfunc
--- a/src/version.c
+++ b/src/version.c
@@ -742,6 +742,8 @@ static char *(features[]) =
 static int included_patches[] =
 {   /* Add new patch number below this line */
 /**/
+    2236,
+/**/
     2235,
 /**/
     2234,