ipif: Use OptionPrefixColon for quitting

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

diff --git a/src/bin/client.rs b/src/bin/client.rs
index 2f2e676ed6b3f266dfbaab59d872fc22a12f0a98..452873305947620ce07156a8a137a15cbbfe5e0c 100644 (file)
--- a/src/bin/client.rs
+++ b/src/bin/client.rs
@@ -292,7 +292,7 @@ async fn run_client<C:HCC>(
     }
   }.await;
 
-  ipif.quitting(&ic).await;
+  ipif.quitting(Some(&ic)).await;
   trouble
 }
 

diff --git a/src/ipif.rs b/src/ipif.rs
index 14f9d6d54346c01faacdd4d0542b47197672fa23..dbffb3a1fbd6e34021039b7a9b3fa6615d65dc57 100644 (file)
--- a/src/ipif.rs
+++ b/src/ipif.rs
@@ -46,22 +46,23 @@ impl Ipif {
     }
   }
 
-  pub async fn quitting(mut self, ic: &InstanceConfig) {
+  pub async fn quitting(mut self, ic: Option<&InstanceConfig>) {
+    let icd = OptionPrefixColon(ic);
     drop(self.tx);
 
     match self.child.wait().await {
-      Err(e) => error!("{}: also, failed to await ipif child: {}", &ic, e),
+      Err(e) => error!("{}also, failed to await ipif child: {}", icd, e),
       Ok(st) => {
         let stderr_timeout = Duration::from_millis(1000);
         match tokio::time::timeout(stderr_timeout, self.stderr_task).await {
           Err::<_,tokio::time::error::Elapsed>(_)
-            => warn!("{}: ipif stderr task continues!", &ic),
-          Ok(Err(e)) => error!("{}: ipif stderr task crashed: {}", &ic, e),
-          Ok(Ok(Err(e))) => error!("{}: ipif stderr read failed: {}", &ic, e),
+            => warn!("{}ipif stderr task continues!", icd),
+          Ok(Err(e)) => error!("{}ipif stderr task crashed: {}", icd, e),
+          Ok(Ok(Err(e))) => error!("{}ipif stderr read failed: {}", icd, e),
           Ok(Ok(Ok(()))) => { },
         }
         if ! st.success() {
-          error!("{}: ipif process failed: {}", &ic, st);
+          error!("{}ipif process failed: {}", icd, st);
         }
       }
     }