diff options
| author | jdrouhard <john@drouhard.dev> | 2024-06-04 10:22:02 -0500 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2024-06-04 17:22:02 +0200 |
| commit | 43581011e41b54473427c2e90450a4f3126b7e66 (patch) | |
| tree | ac9253389310d3bf122d77581c609f40da55bb18 /runtime/lua/vim/lsp/semantic_tokens.lua | |
| parent | 2e6d295f799c27372e5c0c44727fa613c81717fd (diff) | |
| download | rneovim-43581011e41b54473427c2e90450a4f3126b7e66.tar.gz rneovim-43581011e41b54473427c2e90450a4f3126b7e66.tar.bz2 rneovim-43581011e41b54473427c2e90450a4f3126b7e66.zip | |
fix(lsp): remove superfluous on_detach callback from semantic tokens module (#29174)
LspDetach is now triggered by the main on_detach callback that is added
when an LSP client is attached to a buffer. The semantic_tokens module
already includes a LspDetach handler that does the right thing. When the
LspDetach trigger was added to the main LSP on_detach, it created a race
condition in semantic tokens when a buffer was deleted that would
trigger both its own on_detach and the LspDetach handlers. If the former
came last, an error was thrown trying to delete a non-existent augroup
(destroy() was being called twice).
Diffstat (limited to 'runtime/lua/vim/lsp/semantic_tokens.lua')
| -rw-r--r-- | runtime/lua/vim/lsp/semantic_tokens.lua | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/runtime/lua/vim/lsp/semantic_tokens.lua b/runtime/lua/vim/lsp/semantic_tokens.lua index 278014a4ea..b2a8360aa2 100644 --- a/runtime/lua/vim/lsp/semantic_tokens.lua +++ b/runtime/lua/vim/lsp/semantic_tokens.lua @@ -197,12 +197,6 @@ function STHighlighter.new(bufnr) highlighter:send_request() end end, - on_detach = function(_, buf) - local highlighter = STHighlighter.active[buf] - if highlighter then - highlighter:destroy() - end - end, }) api.nvim_create_autocmd({ 'BufWinEnter', 'InsertLeave' }, { |