aboutsummaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorJan Edmund Lazo <jan.lazo@mail.utoronto.ca>2019-05-20 21:56:56 -0400
committerJan Edmund Lazo <jan.lazo@mail.utoronto.ca>2019-05-20 22:48:19 -0400
commit718702078356c4d53ef94e72662450b0971b5056 (patch)
tree0ba49304098f1a447d36c270e2c3a4f18d8f882e /src
parent7c979f972ede6c47b734fe547873848de438ba54 (diff)
downloadrneovim-718702078356c4d53ef94e72662450b0971b5056.tar.gz
rneovim-718702078356c4d53ef94e72662450b0971b5056.tar.bz2
rneovim-718702078356c4d53ef94e72662450b0971b5056.zip
vim-patch:8.1.1360: buffer left 'nomodifiable' after :substitute
Problem: Buffer left 'nomodifiable' after :substitute. (Ingo Karkat) Solution: Save the value of 'modifiable' earlier' (Christian Brabandt, closes vim/vim#4403) https://github.com/vim/vim/commit/80341bcd89764d96f87859a3aac8bc00aad1d762
Diffstat (limited to 'src')
-rw-r--r--src/nvim/ex_cmds.c1
-rw-r--r--src/nvim/testdir/test_substitute.vim18
2 files changed, 18 insertions, 1 deletions
diff --git a/src/nvim/ex_cmds.c b/src/nvim/ex_cmds.c
index 8436ac810e..b3933ac9a6 100644
--- a/src/nvim/ex_cmds.c
+++ b/src/nvim/ex_cmds.c
@@ -3817,6 +3817,7 @@ static buf_T *do_sub(exarg_T *eap, proftime_T timeout,
// 3. Substitute the string. During 'inccommand' preview only do this if
// there is a replace pattern.
if (!preview || has_second_delim) {
+ save_ma = curbuf->b_p_ma;
if (subflags.do_count) {
// prevent accidentally changing the buffer by a function
curbuf->b_p_ma = false;
diff --git a/src/nvim/testdir/test_substitute.vim b/src/nvim/testdir/test_substitute.vim
index 8b306192f0..b29b678129 100644
--- a/src/nvim/testdir/test_substitute.vim
+++ b/src/nvim/testdir/test_substitute.vim
@@ -612,9 +612,24 @@ func Test_sub_replace_10()
call assert_equal('1aaa', substitute('123', '1\zs\|[23]', 'a', 'g'))
endfunc
+func Test_sub_cmd_9()
+ new
+ let input = ['1 aaa', '2 aaa', '3 aaa']
+ call setline(1, input)
+ func Foo()
+ return submatch(0)
+ endfunc
+ %s/aaa/\=Foo()/gn
+ call assert_equal(input, getline(1, '$'))
+ call assert_equal(1, &modifiable)
+
+ delfunc Foo
+ bw!
+endfunc
+
func Test_nocatch_sub_failure_handling()
" normal error results in all replacements
- func! Foo()
+ func Foo()
foobar
endfunc
new
@@ -650,6 +665,7 @@ func Test_nocatch_sub_failure_handling()
call assert_equal(1, error_caught)
call assert_equal(['1 aaa', '2 aaa', '3 aaa'], getline(1, 3))
+ delfunc Foo
bwipe!
endfunc