mirrors/neovim
1
0
Fork 0
mirror of https://github.com/neovim/neovim.git synced 2025-11-28 21:20:45 +00:00
Code Issues Packages Projects Releases Wiki Activity

refactor(log): use msg_schedule_semsg #18950

Browse Source
This commit is contained in:
Justin M. Keyes
2022-06-16 02:23:13 +02:00
committed by GitHub
parent 4cc05a75ea
commit fa4b0c3ba5
2 changed files with 2 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
src/nvim/log.c
View File

@@ -114,14 +114,6 @@ void log_unlock(void)
uv_mutex_unlock(&mutex); uv_mutex_unlock(&mutex);
} }
static void on_log_recursive_event(void **argv)
{
char *fn_name = argv[0];
ptrdiff_t linenr = (ptrdiff_t)argv[1];
siemsg("E5430: %s:%d: recursive log!", fn_name, linenr);
xfree(fn_name);
}
/// Logs a message to $NVIM_LOG_FILE. /// Logs a message to $NVIM_LOG_FILE.
/// ///
/// @param log_level Log level (see log.h) /// @param log_level Log level (see log.h)
@@ -158,9 +150,7 @@ bool logmsg(int log_level, const char *context, const char *func_name, int line_
if (recursive) { if (recursive) {
if (!did_msg) { if (!did_msg) {
did_msg = true; did_msg = true;
char *arg1 = func_name ? xstrdup(func_name) : (context ? xstrdup(context) : NULL); msg_schedule_semsg("E5430: %s:%d: recursive log!", func_name ? func_name : context, line_num);
// coverity[leaked_storage]
loop_schedule_deferred(&main_loop, event_create(on_log_recursive_event, 2, arg1, line_num));
} }
g_stats.log_skip++; g_stats.log_skip++;
log_unlock(); log_unlock();

2
src/nvim/message.c
View File

@@ -853,7 +853,7 @@ void msg_schedule_semsg(const char *const fmt, ...)
va_end(ap); va_end(ap);
char *s = xstrdup((char *)IObuff); char *s = xstrdup((char *)IObuff);
multiqueue_put(main_loop.events, msg_semsg_event, 1, s); loop_schedule_deferred(&main_loop, event_create(msg_semsg_event, 1, s));
} }
/// Like msg(), but truncate to a single line if p_shm contains 't', or when /// Like msg(), but truncate to a single line if p_shm contains 't', or when