neovim

commit 52868977aea32a118fbc548106784c1272491cfe
parent 8b9500c886bdb72620e331d430e166ad7d9c12f8
Author: Maria José Solano <majosolano99@gmail.com>
Date:   Sun, 25 May 2025 15:04:23 -0700

refactor(lsp): use `vim.validate` in `vim.lsp.log` (#34165)


Diffstat:
M
runtime/lua/vim/lsp/log.lua
 | 
10
++++++++--

1 file changed, 8 insertions(+), 2 deletions(-)
diff --git a/runtime/lua/vim/lsp/log.lua b/runtime/lua/vim/lsp/log.lua
@@ -171,11 +171,12 @@ log.warn = create_logger('WARN', log_levels.WARN)
 --- Sets the current log level.
 ---@param level (string|integer) One of |vim.log.levels|
 function log.set_level(level)
+  vim.validate('level', level, { 'string', 'number' })
+
   if type(level) == 'string' then
     current_log_level =
       assert(log.levels[level:upper()], string.format('Invalid log level: %q', level))
   else
-    assert(type(level) == 'number', 'level must be a number or string')
     assert(log.levels[level], string.format('Invalid log level: %d', level))
     current_log_level = level
   end
@@ -190,7 +191,10 @@ end
 --- Sets formatting function used to format logs
 ---@param handle function function to apply to logging arguments, pass vim.inspect for multi-line formatting
 function log.set_format_func(handle)
-  assert(handle == vim.inspect or type(handle) == 'function', 'handle must be a function')
+  vim.validate('handle', handle, function(h)
+    return type(h) == 'function' or h == vim.inspect
+  end, false, 'handle must be a function')
+
   format_func = handle
 end
 
@@ -198,6 +202,8 @@ end
 ---@param level integer log level
 ---@return boolean : true if would log, false if not
 function log.should_log(level)
+  vim.validate('level', level, 'number')
+
   return level >= current_log_level
 end