From e3623f4ecc0bbcab9d0cd2dc001f7493a459c336 Mon Sep 17 00:00:00 2001
From: Ian Wijma <ian@wij.ma>
Date: Sat, 6 Jan 2024 19:57:04 +1100
Subject: [PATCH] Refactor argument handling in main.rs for increased
 readability

The argument handling in main.rs has been adjusted to utilize argument destructuring, making the code more readable and organized. Additionally, the unnecessary new lines were removed, and commas were added to the last elements of the 'Arguments' struct to follow Rust norms and increase consistency.
---
 src/main.rs | 19 ++++++++++---------
 1 file changed, 10 insertions(+), 9 deletions(-)

diff --git a/src/main.rs b/src/main.rs
index 613d9d9..25d02f6 100644
--- a/src/main.rs
+++ b/src/main.rs
@@ -17,7 +17,7 @@ struct Arguments {
     delay: u8,
 
     #[arg()]
-    command: String
+    command: String,
 }
 
 const SECONDS_IN_A_MINUTE: u16 = 60;
@@ -29,7 +29,7 @@ struct Retry {
     max_retries: u8,
     timespan: u16,
     restart_delay: u8,
-    restart_name: String
+    restart_name: String,
 }
 
 fn main() {
@@ -39,22 +39,23 @@ fn main() {
     let mut timespan = SECONDS_IN_A_MINUTE;
     let mut restart_name = String::from("minute");
 
-    if arguments.per_minute > 0 {
-        max_retries = arguments.per_minute;
-    } else if arguments.per_hour > 0 {
-        max_retries = arguments.per_hour;
+    let Arguments { per_minute, per_hour, .. } = arguments;
+    if per_minute > 0 {
+        max_retries = per_minute;
+    } else if per_hour > 0 {
+        max_retries = per_hour;
         timespan = SECONDS_IN_A_HOUR;
         restart_name = String::from("hour");
     }
 
-
+    let Arguments { command, delay: restart_delay, .. } = arguments;
     let retry: Retry = Retry {
         max_retries,
         timespan,
         restart_name,
-        command: arguments.command,
+        command,
+        restart_delay,
         history: Vec::new(),
-        restart_delay: arguments.delay,
     };
 
     run_command(retry)