GitHub/neovim
1
0
Fork 0
mirror of https://github.com/neovim/neovim synced 2025-07-16 09:11:51 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(events): avoid superfluous CursorMovedI on first autocmd (#33588)

Browse Source 
(cherry picked from commit 1dbede5b93)
This commit is contained in:
zeertzjq
2025-04-23 11:20:21 +08:00
committed by github-actions[bot]
parent c633250cc0
commit 155529b91a
2 changed files with 28 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
src/nvim/autocmd.c
View File

@ -1050,9 +1050,10 @@ int autocmd_register(int64_t id, event_T event, const char *pat, int patlen, int
get_mode(last_mode);
}
// If the event is CursorMoved, update the last cursor position
// If the event is CursorMoved or CursorMovedI, update the last cursor position
// position to avoid immediately triggering the autocommand
if (event == EVENT_CURSORMOVED && !has_event(EVENT_CURSORMOVED)) {
if ((event == EVENT_CURSORMOVED && !has_event(EVENT_CURSORMOVED))
|| (event == EVENT_CURSORMOVEDI && !has_event(EVENT_CURSORMOVEDI))) {
last_cursormoved_win = curwin;
last_cursormoved = curwin->w_cursor;
}

26
test/functional/autocmd/cursormoved_spec.lua
View File

@ -7,6 +7,7 @@ local eval = n.eval
local api = n.api
local source = n.source
local command = n.command
local feed = n.feed
describe('CursorMoved', function()
before_each(clear)
@ -49,11 +50,34 @@ describe('CursorMoved', function()
end)
it('is not triggered by cursor movement prior to first CursorMoved instantiation', function()
eq({}, api.nvim_get_autocmds({ event = 'CursorMoved' }))
feed('ifoobar<Esc>')
source([[
let g:cursormoved = 0
autocmd! CursorMoved
autocmd CursorMoved * let g:cursormoved += 1
]])
eq(0, eval('g:cursormoved'))
feed('<Ignore>')
eq(0, eval('g:cursormoved'))
feed('0')
eq(1, eval('g:cursormoved'))
end)
end)
describe('CursorMovedI', function()
before_each(clear)
it('is not triggered by cursor movement prior to first CursorMovedI instantiation', function()
eq({}, api.nvim_get_autocmds({ event = 'CursorMovedI' }))
feed('ifoobar')
source([[
let g:cursormovedi = 0
autocmd CursorMovedI * let g:cursormovedi += 1
]])
eq(0, eval('g:cursormovedi'))
feed('<Ignore>')
eq(0, eval('g:cursormovedi'))
feed('<Home>')
eq(1, eval('g:cursormovedi'))
end)
end)