refactor: simplify duration parsing and remove unnecessary variable assignment

Signed-off-by: Junji Takakura <j.takakura@gmail.com>

Author: jtakakura

crates/wash/src/cli/util.rs

@@ -192,8 +192,6 @@ const fn empty_table_style() -> TableStyle {
 }
 
 mod test {
-    use std::time::Duration;
-
     #[test]
     fn test_safe_base64_parse_option() {
         let base64_option = "config_b64=eyJhZGRyZXNzIjogIjAuMC4wLjA6ODA4MCJ9Cg==".to_string();
@@ -211,25 +209,25 @@ mod test {
         // Test humantime format
         assert_eq!(
             crate::util::parse_duration_fallback_ms("5s").unwrap(),
-            Duration::from_secs(5)
+            std::time::Duration::from_secs(5)
         );
         assert_eq!(
             crate::util::parse_duration_fallback_ms("100ms").unwrap(),
-            Duration::from_millis(100)
+            std::time::Duration::from_millis(100)
         );
         assert_eq!(
             crate::util::parse_duration_fallback_ms("2m").unwrap(),
-            Duration::from_secs(120)
+            std::time::Duration::from_secs(120)
         );
 
         // Test milliseconds fallback
         assert_eq!(
             crate::util::parse_duration_fallback_ms("1000").unwrap(),
-            Duration::from_millis(1000)
+            std::time::Duration::from_millis(1000)
         );
         assert_eq!(
             crate::util::parse_duration_fallback_ms("500").unwrap(),
-            Duration::from_millis(500)
+            std::time::Duration::from_millis(500)
         );
 
         // Test error cases

src/main.rs

@@ -759,13 +759,12 @@ fn parse_duration_fallback_ms(arg: &str) -> anyhow::Result<Duration> {
 /// the argument provided by the user.
 /// This function should be removed once the older duration options are fully deprecated.
 fn get_preferred_arg(old: Duration, new: Duration, default: Duration) -> Duration {
-    let return_value = match (old, new) {
+    match (old, new) {
         (a, b) if a == b => a,
         (a, b) if a == default => b,
         (a, b) if b == default => a,
         _ => default,
-    };
-    return return_value;
+    }
 }
 
 /// Validates that a subject string (e.g. secrets-topic and policy-topic) adheres to the rules and conventions