# HG changeset patch # User Bram Moolenaar # Date 1662480903 -7200 # Node ID eb534258bdaf9efb3e02208b3406ebe0eb4de990 # Parent 80f7a582451b2c35efa31c232c80e9fef813dd10 patch 9.0.0396: :findrepl does not escape '&' and '~' properly Commit: https://github.com/vim/vim/commit/2834ebdee473c838e50e60d0aa160f0e62fc8ef9 Author: matveyt Date: Tue Sep 6 17:00:15 2022 +0100 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 https://github.com/vim/vim/issues/11067) diff --git a/src/gui.c b/src/gui.c --- 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 --- 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 --- a/src/version.c +++ b/src/version.c @@ -704,6 +704,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ /**/ + 396, +/**/ 395, /**/ 394,