patch 9.1.0088: TextChanged not triggered for :norm! commands

Commit: 
https://github.com/vim/vim/commit/c9e79e52845d51f48f5ea3753a62ab3fe0e40184
Author: Christian Brabandt <[email protected]>
Date:   Fri Feb 9 19:34:36 2024 +0100

    patch 9.1.0088: TextChanged not triggered for :norm! commands
    
    Problem:  TextChanged not triggered for :norm! commands
              (machakann, after v9.0.2031)
    Solution: Only reset curbuf->b_last_changedtick if TextChangedI
              was triggered in insert mode (and not blocked)
    
    Note: for unknown reasons, the test fails on Windows (but seems to work
    fine when running interactively)
    
    fixes: #13967
    closes: #13984
    
    Signed-off-by: Christian Brabandt <[email protected]>

diff --git a/src/edit.c b/src/edit.c
index e9a994f6b..5760b06e8 100644
--- a/src/edit.c
+++ b/src/edit.c
@@ -843,7 +843,12 @@ doESCkey:
                if (cmdchar != 'r' && cmdchar != 'v' && c != Ctrl_C)
                    ins_apply_autocmds(EVENT_INSERTLEAVE);
                did_cursorhold = FALSE;
-               curbuf->b_last_changedtick = CHANGEDTICK(curbuf);
+
+               // ins_redraw() triggers TextChangedI only when no characters
+               // are in the typeahead buffer, so only reset 
curbuf->b_last_changedtick
+               // if the TextChangedI was not blocked by char_avail() (e.g. 
using :norm!)
+               if (!char_avail())
+                   curbuf->b_last_changedtick = CHANGEDTICK(curbuf);
                return (c == Ctrl_O);
            }
            continue;
diff --git a/src/testdir/test_autocmd.vim b/src/testdir/test_autocmd.vim
index be73c1819..6e1223275 100644
--- a/src/testdir/test_autocmd.vim
+++ b/src/testdir/test_autocmd.vim
@@ -2706,6 +2706,24 @@ func Test_TextChangedI_with_setline()
   bwipe!
 endfunc
 
+func Test_TextChanged_with_norm()
+  " For unknown reason this fails on MS-Windows
+  CheckNotMSWindows
+  CheckFeature terminal
+  let buf = term_start([GetVimProg(), '--clean', '-c', 'set noswapfile'], 
{'term_rows': 3})
+  call assert_equal('running', term_getstatus(buf))
+  call term_sendkeys(buf, ":let g:a=0\<cr>")
+  call term_wait(buf, 50)
+  call term_sendkeys(buf, ":au! TextChanged * :let g:a+=1\<cr>")
+  call term_wait(buf, 50)
+  call term_sendkeys(buf, ":norm! ia\<cr>")
+  call term_wait(buf, 50)
+  call term_sendkeys(buf, ":echo g:a\<cr>")
+  call term_wait(buf, 50)
+  call WaitForAssert({-> assert_match('^1.*$', term_getline(buf, 3))})
+  bwipe!
+endfunc
+
 func Test_Changed_FirstTime()
   CheckFeature terminal
   CheckNotGui
diff --git a/src/version.c b/src/version.c
index 08a78b600..1332f46ba 100644
--- 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 */
+/**/
+    88,
 /**/
     87,
 /**/

-- 
-- 
You received this message from the "vim_dev" maillist.
Do not top-post! Type your reply below the text you are replying to.
For more information, visit http://www.vim.org/maillist.php

--- 
You received this message because you are subscribed to the Google Groups 
"vim_dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To view this discussion on the web visit 
https://groups.google.com/d/msgid/vim_dev/E1rYVs1-005ZeO-52%40256bit.org.

Raspunde prin e-mail lui