mirrors/neovim
1
0
Fork 0
mirror of https://github.com/neovim/neovim.git synced 2026-07-20 08:01:46 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(messages): increment message ID without ext_messages #37714

Browse Source 
Problem:  Message ID is not incremented without ext_messages.
Solution: Increment where callstack reaches without ext_messages.
          There is no clear place marking the end of internal
          messages without ext_messages so IDs are only incremented
          with `nvim_echo` calls. Message IDs are currently not exposed
          anywhere but the msg_show event for this to matter.
This commit is contained in:
luukvbaal
2026-02-05 15:55:11 +01:00
committed by GitHub
parent 13a9cdc6b4
commit 7a490d65c5
2 changed files with 9 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
src/nvim/message.c
View File

@@ -365,10 +365,10 @@ MsgID msg_multihl(MsgID id, HlMessage hl_msg, const char *kind, bool history, bo
// provide a new id if not given // provide a new id if not given
if (id.type == kObjectTypeNil) { if (id.type == kObjectTypeNil) {
id = INTEGER_OBJ(msg_id_next); id = INTEGER_OBJ(msg_id_next++);
} else if (id.type == kObjectTypeInteger) { } else if (id.type == kObjectTypeInteger) {
id = id.data.integer > 0 ? id : INTEGER_OBJ(msg_id_next); id = id.data.integer > 0 ? id : INTEGER_OBJ(msg_id_next++);
msg_id_next = MAX(msg_id_next, id.data.integer); msg_id_next = MAX(msg_id_next, id.data.integer + 1);
} }
msg_ext_id = id; msg_ext_id = id;

6
test/functional/api/vim_spec.lua
View File

@@ -3853,6 +3853,12 @@ describe('API', function()
{9:Error}{16:Message} | {9:Error}{16:Message} |
]]) ]])
end) end)
it('increments message ID', function()
eq(1, api.nvim_echo({ { 'foo' } }, false, {}))
eq(4, api.nvim_echo({ { 'foo' } }, false, { id = 4 }))
eq(5, api.nvim_echo({ { 'foo' } }, false, {}))
end)
end) end)
describe('nvim_open_term', function() describe('nvim_open_term', function()