TudbuT/helix-mods
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

workspace symbols: Default to empty Vec on None

Browse source 
A language server might send None as the response to workspace symbols.
We should treat this as the empty Vec rather than the server sending
an error status. This fixes the interaction with gopls which uses
None to mean no matching symbols.
This commit is contained in:
Michael Davis 2022-12-08 19:54:15 -06:00 • committed by Blaž Hrastnik
parent 35cf972ce4
commit 2a60de74f9
1 changed files with 2 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
helix-term/src/commands/lsp.rs
View file

@ -386,10 +386,7 @@ pub fn workspace_symbol_picker(cx: &mut Context) {
cx.callback( cx.callback(
future, future,
move |_editor, compositor, response: Option<Vec<lsp::SymbolInformation>>| { move |_editor, compositor, response: Option<Vec<lsp::SymbolInformation>>| {
let symbols = match response { let symbols = response.unwrap_or_default();
Some(s) => s,
None => return,
};
let picker = sym_picker(symbols, current_url, offset_encoding); let picker = sym_picker(symbols, current_url, offset_encoding);
let get_symbols = |query: String, editor: &mut Editor| { let get_symbols = |query: String, editor: &mut Editor| {
let doc = doc!(editor); let doc = doc!(editor);
@ -420,9 +417,7 @@ pub fn workspace_symbol_picker(cx: &mut Context) {
let response: Option<Vec<lsp::SymbolInformation>> = let response: Option<Vec<lsp::SymbolInformation>> =
serde_json::from_value(json)?; serde_json::from_value(json)?;
response.ok_or_else(|| { Ok(response.unwrap_or_default())
anyhow::anyhow!("No response for workspace symbols from language server")
})
}; };
future.boxed() future.boxed()
}; };