From 176fe4658b6c03b0dce1f2d2d1431090db31473f Mon Sep 17 00:00:00 2001
From: Lars Wirzenius <liw@liw.fi>
Date: Tue, 28 Sep 2021 20:24:25 +0300
Subject: Be explicit about giving Ok() the unit value

Instead of giving Ok() the return value of a function that returns no
value, i.e., the unit value (), give it explicitly. This is clearer
and easier to follow for the reader.

Found by clippy lint unit_arg:
https://rust-lang.github.io/rust-clippy/master/index.html#unit_arg
---
 ipc/src/lib.rs | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

(limited to 'ipc')

diff --git a/ipc/src/lib.rs b/ipc/src/lib.rs
index 3f1c1138..734725bb 100644
--- a/ipc/src/lib.rs
+++ b/ipc/src/lib.rs
@@ -287,10 +287,11 @@ impl Descriptor {
     fn spawn(&self, l: TcpListener) -> Result<()> {
         let descriptor = self.clone();
         thread::spawn(move || -> Result<()> {
-            Ok(Server::new(descriptor)
+            Server::new(descriptor)
                .expect("Failed to spawn server") // XXX
                .serve_listener(l)
-               .expect("Failed to spawn server")) // XXX
+               .expect("Failed to spawn server"); // XXX
+            Ok(())
         });
         Ok(())
     }
-- 
cgit v1.2.3