From e33685f77b5de5dd1d45a09d6e15c50c1afe7a94 Mon Sep 17 00:00:00 2001
From: Lars Wirzenius <liw@liw.fi>
Date: Sun, 28 Oct 2018 19:17:13 +0200
Subject: Refactor: how the right function gets called for a subcommand

---
 src/main.rs | 35 +++++++++++++++++++++++------------
 1 file changed, 23 insertions(+), 12 deletions(-)

diff --git a/src/main.rs b/src/main.rs
index ccae56f..eb71ec6 100644
--- a/src/main.rs
+++ b/src/main.rs
@@ -10,20 +10,31 @@ fn main() {
         println!("api: {}", api);
     }
 
-    if let Some(_) = matches.subcommand_matches("version") {
-        version();
-    } else if let Some(_) = matches.subcommand_matches("status") {
-        status();
-    } else {
-        eprintln!("no action taken!");
-        std::process::exit(1);
-    }
+    let code = match matches.subcommand_name() {
+        Some("version") => version(),
+        Some("status") => status(),
+        None => none(),
+        _ => unknown(),
+    };
+    std::process::exit(code);
+}
+
+fn none() -> i32 {
+    eprintln!("No subcommand was given");
+    1
+}
+
+fn unknown() -> i32 {
+    eprintln!("Unknown command");
+    1
 }
 
-fn version() {
-        println!("status requested");
+fn version() -> i32 {
+    println!("status requested");
+    0
 }
 
-fn status() {
-        println!("status requested");
+fn status() -> i32 {
+    println!("status requested");
+    0
 }
-- 
cgit v1.2.1