helix

Commit Graph

Author	SHA1	Message	Date
Michael Davis	5532ef35d9	LSP: Remove future wrapper from `Client::notify`, `Client::reply` Previously LSP notifications were sent within a future and most callers used a `tokio::spawn` to send the notification, silently discarding any failures like problems serializing parameters or sending on the channel. It's possible that tokio could schedule futures out of intended order though which could cause notifications where order is important, like document synchronization, to become partially shuffled. This change removes the future wrapper and logs all internal failures. Also included in this commit is the same change for `Client::reply` which was also unnecessarily wrapped in a future. Co-authored-by: Pascal Kuthe <pascalkuthe@pm.me>	2025-02-01 19:32:37 -05:00
Michael Davis	9dd51e75e0	Resolve new Clippy lints	2024-05-07 15:15:52 -04:00
Pascal Kuthe	b834806dbc	use newtype parttern for langauge server id	2024-04-22 12:27:47 +09:00
Jonas Tepe	505213d41b	Drop mut from variable to silence linter (#7704 )	2023-07-22 23:31:24 +09:00
Ryan Fowler	5c41f22c2a	Add support for LSP DidChangeWatchedFiles (#7665 ) * Add initial support for LSP DidChangeWatchedFiles * Move file event Handler to helix-lsp * Simplify file event handling * Refactor file event handling * Block on future within LSP file event handler * Fully qualify uses of the file_event::Handler type * Rename ops field to options * Revert newline removal from helix-view/Cargo.toml * Ensure file event Handler is cleaned up when lsp client is shutdown	2023-07-22 00:21:21 +02:00

5 Commits (8c5dd7f0c37cbf54a6055916ba45be3cf7dd2340)