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

when text document sync capability is only kind send didSave without text (#9101)

Browse source 
see https://github.com/microsoft/language-server-protocol/issues/288 for details
This commit is contained in:
Gabriel Dinner-David 2023-12-17 16:59:04 -05:00 • committed by GitHub
parent a1a20d231f
commit f27fdb2bf4
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 7 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
helix-lsp/src/client.rs
View file

@ -969,20 +969,19 @@ impl Client {
) -> Option<impl Future<Output = Result<()>>> { ) -> Option<impl Future<Output = Result<()>>> {
let capabilities = self.capabilities.get().unwrap(); let capabilities = self.capabilities.get().unwrap();
let include_text = match &capabilities.text_document_sync { let include_text = match &capabilities.text_document_sync.as_ref()? {
Some(lsp::TextDocumentSyncCapability::Options(lsp::TextDocumentSyncOptions { lsp::TextDocumentSyncCapability::Options(lsp::TextDocumentSyncOptions {
save: Some(options), save: options,
.. ..
})) => match options { }) => match options.as_ref()? {
lsp::TextDocumentSyncSaveOptions::Supported(true) => false, lsp::TextDocumentSyncSaveOptions::Supported(true) => false,
lsp::TextDocumentSyncSaveOptions::SaveOptions(lsp_types::SaveOptions { lsp::TextDocumentSyncSaveOptions::SaveOptions(lsp_types::SaveOptions {
include_text, include_text,
}) => include_text.unwrap_or(false), }) => include_text.unwrap_or(false),
// Supported(false) lsp::TextDocumentSyncSaveOptions::Supported(false) => return None,
_ => return None,
}, },
// unsupported // see: https://github.com/microsoft/language-server-protocol/issues/288
_ => return None, lsp::TextDocumentSyncCapability::Kind(..) => false,
}; };
Some(self.notify::<lsp::notification::DidSaveTextDocument>( Some(self.notify::<lsp::notification::DidSaveTextDocument>(