diff options
| author | dundargoc <gocdundar@gmail.com> | 2024-01-22 18:23:28 +0100 |
|---|---|---|
| committer | dundargoc <33953936+dundargoc@users.noreply.github.com> | 2024-02-03 16:53:41 +0100 |
| commit | 2e982f1aad9f1a03562b7a451d642f76b04c37cb (patch) | |
| tree | 7f689e027d93092a6a1c5f783ff0e4909b2ecb9d /runtime/lua/vim/diagnostic.lua | |
| parent | 51702e0aea99fddba74e299e2640dd350a348df3 (diff) | |
| download | rneovim-2e982f1aad9f1a03562b7a451d642f76b04c37cb.tar.gz rneovim-2e982f1aad9f1a03562b7a451d642f76b04c37cb.tar.bz2 rneovim-2e982f1aad9f1a03562b7a451d642f76b04c37cb.zip | |
refactor: create function for deferred loading
The benefit of this is that users only pay for what they use. If e.g.
only `vim.lsp.buf_get_clients()` is called then they don't need to load
all modules under `vim.lsp` which could lead to significant startuptime
saving.
Also `vim.lsp.module` is a bit nicer to user compared to
`require("vim.lsp.module")`.
This isn't used for some nested modules such as `filetype` as it breaks
tests with error messages such as "attempt to index field 'detect'".
It's not entirely certain the reason for this, but it is likely it is
due to filetype being precompiled which would imply deferred loading
isn't needed for performance reasons.
Diffstat (limited to 'runtime/lua/vim/diagnostic.lua')
| -rw-r--r-- | runtime/lua/vim/diagnostic.lua | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/runtime/lua/vim/diagnostic.lua b/runtime/lua/vim/diagnostic.lua index 8845fb5b61..a3a2422ab5 100644 --- a/runtime/lua/vim/diagnostic.lua +++ b/runtime/lua/vim/diagnostic.lua @@ -1732,7 +1732,7 @@ function M.open_float(opts, ...) if not opts.focus_id then opts.focus_id = scope end - local float_bufnr, winnr = require('vim.lsp.util').open_floating_preview(lines, 'plaintext', opts) + local float_bufnr, winnr = vim.lsp.util.open_floating_preview(lines, 'plaintext', opts) for i, hl in ipairs(highlights) do local line = lines[i] local prefix_len = hl.prefix and hl.prefix.length or 0 |