Do not crash if build process exits early

See #837.
latex-lsp · Jan 18, 2023 · 1a47c8c · 1a47c8c
1 parent f60e9bd
commit 1a47c8c
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 11 deletions.
diff --git a/src/features/build.rs b/src/features/build.rs
@@ -127,7 +127,7 @@ impl Command {
                     .recv(&line_receiver, |line| match line {
                         Ok(message) => {
                             let params = LogMessageParams { message, typ };
-                            client.send_notification::<LogMessage>(params).unwrap();
+                            let _ = client.send_notification::<LogMessage>(params);
                             false
                         }
                         Err(_) => true,
@@ -170,9 +170,10 @@ fn track_output(
     );
 
     thread::spawn(move || {
-        for line in reader.lines() {
-            sender.send(line.unwrap()).unwrap();
-        }
+        let _ = reader
+            .lines()
+            .flatten()
+            .try_for_each(|line| sender.send(line));
     })
 }
 

diff --git a/src/server.rs b/src/server.rs
@@ -678,9 +678,7 @@ impl Server {
         let client = self.client.clone();
         self.build_internal(uri, move |status| {
             let result = BuildResult { status };
-            client
-                .send_response(lsp_server::Response::new_ok(id, result))
-                .unwrap();
+            let _ = client.send_response(lsp_server::Response::new_ok(id, result));
         })?;
 
         Ok(())
@@ -710,7 +708,7 @@ impl Server {
 
             let status = compiler.run();
             if forward_search_after {
-                sender.send(InternalMessage::ForwardSearch(uri)).unwrap();
+                let _ = sender.send(InternalMessage::ForwardSearch(uri));
             }
 
             drop(guard);
@@ -727,9 +725,7 @@ impl Server {
         let client = self.client.clone();
         self.forward_search_internal(uri, Some(params.position), move |status| {
             let result = ForwardSearchResult { status };
-            client
-                .send_response(lsp_server::Response::new_ok(id, result))
-                .unwrap();
+            let _ = client.send_response(lsp_server::Response::new_ok(id, result));
         })?;
 
         Ok(())