mirrors/neovim
1
0
Fork 0
mirror of https://github.com/neovim/neovim.git synced 2025-10-01 23:48:32 +00:00
Code Issues Packages Projects Releases Wiki Activity

perf(events): skip Progress prep if there are no subscribers #35871

Browse Source 
Problem:
`do_autocmd_progress` does unnecessary work even if there is no Progress
event handler defined.

Solution:
Check `has_event`.
This commit is contained in:
Justin M. Keyes
2025-09-21 21:13:46 -04:00
committed by GitHub
parent e887602eb5
commit 4adfc4b7bc
1 changed files with 5 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
src/nvim/message.c
View File

@@ -1108,6 +1108,10 @@ static bool do_clear_hist_temp = true;
void do_autocmd_progress(MsgID msg_id, HlMessage msg, MessageData *msg_data)
{
if (!has_event(EVENT_PROGRESS)) {
return;
}
MAXSIZE_TEMP_DICT(data, 7);
ArrayOf(String) messages = ARRAY_DICT_INIT;
for (size_t i = 0; i < msg.size; i++) {
@@ -1124,8 +1128,7 @@ void do_autocmd_progress(MsgID msg_id, HlMessage msg, MessageData *msg_data)
}
apply_autocmds_group(EVENT_PROGRESS, msg_data ? msg_data->title.data : "", NULL, true,
AUGROUP_ALL, NULL,
NULL, &DICT_OBJ(data));
AUGROUP_ALL, NULL, NULL, &DICT_OBJ(data));
kv_destroy(messages);
}