diff options
| author | Andrew Braxton <andrewcbraxton@gmail.com> | 2026-02-23 14:15:53 -0500 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2026-02-23 11:15:53 -0800 |
| commit | eb90f5d9e3a6615870f9bf0b28e30f6e84ccd6f4 (patch) | |
| tree | 49cc79289b895c7da90f11593bfb19dca918ec99 /test/functional/vimscript/executable_spec.lua | |
| parent | 4d754d270456673f5fc4a1ece1cd5299cbffb6f0 (diff) | |
| download | rneovim-eb90f5d9e3a6615870f9bf0b28e30f6e84ccd6f4.tar.gz rneovim-eb90f5d9e3a6615870f9bf0b28e30f6e84ccd6f4.tar.bz2 rneovim-eb90f5d9e3a6615870f9bf0b28e30f6e84ccd6f4.zip | |
fix(lsp): only resolve LSP configs once (#38007)
`lsp.config[]` resolves an LSP config the first time it is called, and
returns the cached result on subsequent calls.
The change in #37571 added an extra call to `lsp.config[]` which will
resolve the config *before* the server is added to `_enabled_configs`,
meaning the result is discarded. That means configs will be needlessly
resolved again once `lsp_enable_callback` fires for the first time. That
includes an additional `loadfile()` call which is relatively expensive
and can have unexpected side effects.
Avoid this by storing the result of the initial call to `lsp.config[]`
in `_enabled_configs` so the config is not resolved a second time once
`lsp_enable_callback` is called for the first time.
Diffstat (limited to 'test/functional/vimscript/executable_spec.lua')
0 files changed, 0 insertions, 0 deletions