From 92c527b61043e37320c2cdf1b46ea96e44ce49f9 Mon Sep 17 00:00:00 2001
From: Lars Wirzenius <liw@liw.fi>
Date: Mon, 1 Mar 2021 09:25:13 +0200
Subject: fix: simplify checking for success

---
 src/bin/vmadm.rs | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/src/bin/vmadm.rs b/src/bin/vmadm.rs
index 75e0cdf..a2717ee 100644
--- a/src/bin/vmadm.rs
+++ b/src/bin/vmadm.rs
@@ -88,9 +88,8 @@ fn new(spec: &Path) -> anyhow::Result<()> {
 fn wait_for_port(name: &str, port: i32) -> anyhow::Result<()> {
     let addr = format!("{}:{}", name, port);
     loop {
-        match TcpStream::connect(&addr) {
-            Ok(_) => return Ok(()),
-            Err(_) => (),
+        if TcpStream::connect(&addr).is_ok() {
+            return Ok(());
         }
     }
 }
@@ -150,7 +149,6 @@ fn delete(spec: &Path) -> anyhow::Result<()> {
                     Ok(false) => break,
                     Err(err) => {
                         debug!("is_active: {}", err);
-                        ()
                     }
                 }
                 debug!("{} is still running", spec.name);
-- 
cgit v1.2.1