From 2834ebdee473c838e50e60d0aa160f0e62fc8ef9 Mon Sep 17 00:00:00 2001 From: matveyt Date: Tue, 6 Sep 2022 17:00:15 +0100 Subject: [PATCH] patch 9.0.0396: :findrepl does not escape '&' and '~' properly Problem: :findrepl does not escape '&' and '~' properly. Solution: Escape depending on the value of 'magic'. (closes #11067) --- src/gui.c | 6 ++++-- src/testdir/test_gui.vim | 6 ++++++ src/version.c | 2 ++ 3 files changed, 12 insertions(+), 2 deletions(-) diff --git a/src/gui.c b/src/gui.c index 9c78dacb1..b9217b459 100644 --- a/src/gui.c +++ b/src/gui.c @@ -5360,8 +5360,10 @@ gui_do_findrepl( if (type == FRD_REPLACEALL) { ga_concat(&ga, (char_u *)"/"); - // escape slash and backslash - p = vim_strsave_escaped(repl_text, (char_u *)"/\\"); + // Escape slash and backslash. + // Also escape tilde and ampersand if 'magic' is set. + p = vim_strsave_escaped(repl_text, + p_magic ? (char_u *)"/\\~&" : (char_u *)"/\\"); if (p != NULL) ga_concat(&ga, p); vim_free(p); diff --git a/src/testdir/test_gui.vim b/src/testdir/test_gui.vim index 29e4ecdd5..a2491458d 100644 --- a/src/testdir/test_gui.vim +++ b/src/testdir/test_gui.vim @@ -1580,6 +1580,12 @@ func Test_gui_findrepl() call test_gui_event('findrepl', args) call assert_equal(['ONE two ONE', 'Twoo ONE two ONEo'], getline(1, '$')) + " Replace all instances with sub-replace specials + call cursor(1, 1) + let args = #{find_text: 'ONE', repl_text: '&~&', flags: 0x4, forward: 1} + call test_gui_event('findrepl', args) + call assert_equal(['&~& two &~&', 'Twoo &~& two &~&o'], getline(1, '$')) + " Invalid arguments call assert_false(test_gui_event('findrepl', {})) let args = #{repl_text: 'a', flags: 1, forward: 1} diff --git a/src/version.c b/src/version.c index 2cf848c63..f1170220c 100644 --- a/src/version.c +++ b/src/version.c @@ -703,6 +703,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ +/**/ + 396, /**/ 395, /**/ -- 2.40.0