From 570268f750000a99a811af090aaa0284463ac05a Mon Sep 17 00:00:00 2001
From: Daniel Hofstetter <daniel.hofstetter@42dh.com>
Date: Sat, 13 Jun 2026 07:53:14 +0200
Subject: [PATCH] dircolors: replace if/else chain with match

---
 src/uu/dircolors/src/dircolors.rs | 69 +++++++++++++++----------------
 1 file changed, 34 insertions(+), 35 deletions(-)

diff --git a/src/uu/dircolors/src/dircolors.rs b/src/uu/dircolors/src/dircolors.rs
index 55e2b17872..de842fea87 100644
--- a/src/uu/dircolors/src/dircolors.rs
+++ b/src/uu/dircolors/src/dircolors.rs
@@ -163,44 +163,43 @@ pub fn uumain(args: impl uucore::Args) -> UResult<()> {
             })?
     };
 
-    let result;
-    if files.is_empty() {
-        writeln!(stdout(), "{}", generate_ls_colors(&out_format, ":"))?;
-        return Ok(());
-    } else if files.len() > 1 {
-        return Err(UUsageError::new(
+    match files.as_slice() {
+        [] => {
+            writeln!(stdout(), "{}", generate_ls_colors(&out_format, ":"))?;
+            Ok(())
+        }
+        [_file_arg, extra, ..] => Err(UUsageError::new(
             1,
-            translate!("dircolors-error-extra-operand", "operand" => files[1].quote()),
-        ));
-    } else if files[0] == "-" {
-        let fin = BufReader::new(std::io::stdin());
-        // For example, for echo "owt 40;33"|dircolors -b -
-        result = parse(
-            fin.lines().map_while(Result::ok),
-            &out_format,
-            &files[0].to_string_lossy(),
-        );
-    } else {
-        let path = Path::new(&files[0]);
-        if path.is_dir() {
-            return Err(USimpleError::new(
-                2,
-                translate!("dircolors-error-expected-file-got-directory", "path" => path.quote()),
-            ));
+            translate!("dircolors-error-extra-operand", "operand" => extra.quote()),
+        )),
+        [file_arg] => {
+            let result = if *file_arg == "-" {
+                let fin = BufReader::new(std::io::stdin());
+                // For example, for echo "owt 40;33"|dircolors -b -
+                parse(fin.lines().map_while(Result::ok), &out_format, "-")
+            } else {
+                let path = Path::new(&file_arg);
+                if path.is_dir() {
+                    return Err(USimpleError::new(
+                        2,
+                        translate!("dircolors-error-expected-file-got-directory", "path" => path.quote()),
+                    ));
+                }
+                let file = File::open(path)
+                    .map_err(|e| USimpleError::new(1, format!("{}: {e}", path.maybe_quote())))?;
+                let fin = BufReader::new(file);
+                parse(
+                    fin.lines().map_while(Result::ok),
+                    &out_format,
+                    &path.to_string_lossy(),
+                )
+            };
+
+            let string = result.map_err(|s| USimpleError::new(1, s))?;
+            writeln!(stdout(), "{string}")?;
+            Ok(())
         }
-        let file = File::open(path)
-            .map_err(|e| USimpleError::new(1, format!("{}: {e}", path.maybe_quote())))?;
-        let fin = BufReader::new(file);
-        result = parse(
-            fin.lines().map_while(Result::ok),
-            &out_format,
-            &path.to_string_lossy(),
-        );
     }
-
-    let string = result.map_err(|s| USimpleError::new(1, s))?;
-    writeln!(stdout(), "{string}")?;
-    Ok(())
 }
 
 pub fn uu_app() -> Command {