diff --git a/src/uu/unexpand/src/unexpand.rs b/src/uu/unexpand/src/unexpand.rs index 840858c6519..b817dcdff93 100644 --- a/src/uu/unexpand/src/unexpand.rs +++ b/src/uu/unexpand/src/unexpand.rs @@ -115,11 +115,9 @@ fn parse_tabstops(s: &str) -> Result { // Handle the increment if specified // Only add an extra tab stop if increment is non-zero - if let Some(inc) = increment_size { - if inc > 0 { - let last = *nums.last().unwrap(); - nums.push(last + inc); - } + if let Some(inc) = increment_size.filter(|&i| i > 0) { + let last = *nums.last().unwrap(); + nums.push(last + inc); } if let (false, _) = nums diff --git a/src/uucore/src/lib/features/fs.rs b/src/uucore/src/lib/features/fs.rs index aece06bd604..351a29f9df8 100644 --- a/src/uucore/src/lib/features/fs.rs +++ b/src/uucore/src/lib/features/fs.rs @@ -633,12 +633,7 @@ pub fn infos_refer_to_same_file( info1: IOResult, info2: IOResult, ) -> bool { - if let Ok(info1) = info1 { - if let Ok(info2) = info2 { - return info1 == info2; - } - } - false + info1.is_ok() && info1.ok() == info2.ok() } /// Converts absolute `path` to be relative to absolute `to` path.