lsp: Do not error out on requests returning an empty response

It is perfectly valid for Requests to return `()`.
slint-ui · Nov 29, 2022 · 01a9309 · 01a9309
1 parent b48c5a4
commit 01a9309
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/tools/lsp/main.rs b/tools/lsp/main.rs
@@ -94,13 +94,11 @@ impl ServerNotifier {
                 Some(OutgoingRequest::Done(d)) => {
                     if let Some(err) = d.error {
                         Poll::Ready(Err(err.message.into()))
-                    } else if let Some(d) = d.result {
+                    } else {
                         Poll::Ready(
-                            serde_json::from_value(d)
+                            serde_json::from_value(d.result.unwrap_or_default())
                                 .map_err(|e| format!("cannot deserialize response: {e:?}").into()),
                         )
-                    } else {
-                        Poll::Ready(Err("No response".into()))
                     }
                 }
             }