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

1 files changed, 3 insertions, 2 deletions

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(())
     }