Unnamed repository; edit this file 'description' to name the repository.
Diffstat (limited to 'lib/lsp-server/src/stdio.rs')
-rw-r--r--lib/lsp-server/src/stdio.rs46
1 files changed, 16 insertions, 30 deletions
diff --git a/lib/lsp-server/src/stdio.rs b/lib/lsp-server/src/stdio.rs
index 711c4e9907..d14d8ee55d 100644
--- a/lib/lsp-server/src/stdio.rs
+++ b/lib/lsp-server/src/stdio.rs
@@ -3,41 +3,35 @@ use std::{
thread,
};
-use log::debug;
+use log::{debug, trace};
-use crossbeam_channel::{Receiver, Sender, bounded};
+use crossbeam_channel::{Receiver, Sender, bounded, unbounded};
use crate::Message;
-
/// Creates an LSP connection via stdio.
-pub(crate) fn stdio_transport(
+pub fn stdio_transport(
mut read_from: impl Read + std::io::BufRead + Sync + Send + 'static,
mut write_to: impl Write + Sync + Send + 'static,
) -> (Sender<Message>, Receiver<Message>, IoThreads) {
- let (drop_sender, drop_receiver) = bounded::<Message>(0);
- let (writer_sender, writer_receiver) = bounded::<Message>(0);
+ let (writer_sender, writer_receiver) = unbounded::<Message>();
let writer = thread::Builder::new()
- .name("LspServerWriter".to_owned())
+ .name("send to lsp".to_owned())
.spawn(move || {
- writer_receiver.into_iter().try_for_each(|it| {
- let result = it.write(&mut write_to);
- let _ = drop_sender.send(it);
+ loop {
+ let it = writer_receiver.recv().unwrap();
+ trace!("sent message {it:#?}");
+ let result = it.write(&mut write_to).unwrap();
result
- })
+ }
})
.unwrap();
- let dropper = thread::Builder::new()
- .name("LspMessageDropper".to_owned())
- .spawn(move || drop_receiver.into_iter().for_each(drop))
- .unwrap();
let (reader_sender, reader_receiver) = bounded::<Message>(0);
let reader: thread::JoinHandle<Result<(), io::Error>> = thread::Builder::new()
- .name("LspServerReader".to_owned())
+ .name("read from lsp".to_owned())
.spawn(move || {
while let Some(msg) = Message::read(&mut read_from)? {
let is_exit = matches!(&msg, Message::Notification(n) if n.is_exit());
-
- debug!("sending message {msg:#?}");
+ trace!("received message {msg:#?}");
if let Err(e) = reader_sender.send(msg) {
return Err(io::Error::other(e));
}
@@ -49,7 +43,7 @@ pub(crate) fn stdio_transport(
Ok(())
})
.unwrap();
- let threads = IoThreads { reader, writer, dropper };
+ let threads = IoThreads { reader, writer };
(writer_sender, reader_receiver, threads)
}
@@ -57,15 +51,13 @@ pub(crate) fn stdio_transport(
pub(crate) fn make_io_threads(
reader: thread::JoinHandle<io::Result<()>>,
writer: thread::JoinHandle<io::Result<()>>,
- dropper: thread::JoinHandle<()>,
) -> IoThreads {
- IoThreads { reader, writer, dropper }
+ IoThreads { reader, writer }
}
pub struct IoThreads {
- reader: thread::JoinHandle<io::Result<()>>,
- writer: thread::JoinHandle<io::Result<()>>,
- dropper: thread::JoinHandle<()>,
+ pub reader: thread::JoinHandle<io::Result<()>>,
+ pub writer: thread::JoinHandle<io::Result<()>>,
}
impl IoThreads {
@@ -74,12 +66,6 @@ impl IoThreads {
Ok(r) => r?,
Err(err) => std::panic::panic_any(err),
}
- match self.dropper.join() {
- Ok(_) => (),
- Err(err) => {
- std::panic::panic_any(err);
- }
- }
match self.writer.join() {
Ok(r) => r,
Err(err) => {