From ecd3cdba8095ce2aba002cfbd3428c7c55c720e4 Mon Sep 17 00:00:00 2001 From: Daniel Hofstetter Date: Tue, 30 Jul 2024 09:22:24 +0200 Subject: [PATCH] rev: use "if let" instead of match --- src/uu/rev/src/rev.rs | 30 ++++++++++++++---------------- 1 file changed, 14 insertions(+), 16 deletions(-) diff --git a/src/uu/rev/src/rev.rs b/src/uu/rev/src/rev.rs index 9cf1f37..b980d16 100644 --- a/src/uu/rev/src/rev.rs +++ b/src/uu/rev/src/rev.rs @@ -24,24 +24,22 @@ pub fn uumain(args: impl uucore::Args) -> UResult<()> { let zero = matches.get_flag(options::ZERO); let sep = if zero { b'\0' } else { b'\n' }; - match files { - Some(files) => { - for path in files { - let Ok(file) = std::fs::File::open(path) else { - uucore::error::set_exit_code(1); - uucore::show_error!("cannot open {path}: No such file or directory"); - continue; - }; - if let Err(err) = rev_stream(file, sep) { - uucore::error::set_exit_code(1); - uucore::show_error!("cannot read {path}: {err}"); - } + + if let Some(files) = files { + for path in files { + let Ok(file) = std::fs::File::open(path) else { + uucore::error::set_exit_code(1); + uucore::show_error!("cannot open {path}: No such file or directory"); + continue; + }; + if let Err(err) = rev_stream(file, sep) { + uucore::error::set_exit_code(1); + uucore::show_error!("cannot read {path}: {err}"); } } - None => { - let stdin = std::io::stdin().lock(); - let _ = rev_stream(stdin, sep); - } + } else { + let stdin = std::io::stdin().lock(); + let _ = rev_stream(stdin, sep); } Ok(())