Merge pull request #354 from cakebaker/clippy_fix_warnings_rust_1_89

clippy: fix warnings introduced with Rust `1.89`

Author: sylvestre

src/uu/chcpu/src/sysfs.rs

@@ -217,10 +217,12 @@ impl SysFSCpu {
                 .map_err(|err| ChCpuError::io0("write standard output", err));
         }
 
-        if let Some(enabled_cpu_list) = enabled_cpu_list {
-            if previous_config && !configure && enabled_cpu_list.0.contains(&cpu_index) {
-                return Err(ChCpuError::CpuIsEnabled(cpu_index));
-            }
+        if let Some(enabled_cpu_list) = enabled_cpu_list
+            && previous_config
+            && !configure
+            && enabled_cpu_list.0.contains(&cpu_index)
+        {
+            return Err(ChCpuError::CpuIsEnabled(cpu_index));
         }
 
         if let Err(err) = self.write_value(&configure_path, u8::from(configure)) {

src/uu/lsipc/src/display.rs

@@ -246,7 +246,10 @@ pub(crate) fn describe_size(size: u64, in_bytes: bool) -> Option<Cow<'static, CS
     Some(Cow::Owned(CString::new(size_desc(size, in_bytes)).unwrap()))
 }
 
-pub(crate) fn describe_owner(users: &mut UserDbRecordRef, uid: libc::uid_t) -> Option<Cow<CStr>> {
+pub(crate) fn describe_owner(
+    users: &mut UserDbRecordRef,
+    uid: libc::uid_t,
+) -> Option<Cow<'_, CStr>> {
     if let Some(name) = users.for_id(uid).name() {
         Some(Cow::Borrowed(name))
     } else {

src/uu/lsipc/src/message_queue.rs

@@ -341,11 +341,11 @@ pub(crate) fn describe(
     let now = time_of_day()?;
     let sys_v_ipc = SysVIpc::new(id)?;
 
-    if let Some(id) = id {
-        if sys_v_ipc.0.len() != 1 {
-            eprintln!("id {id} not found");
-            return Ok(());
-        }
+    if let Some(id) = id
+        && sys_v_ipc.0.len() != 1
+    {
+        eprintln!("id {id} not found");
+        return Ok(());
     }
 
     table.set_name(c"messages")?;

src/uu/lsipc/src/semaphore.rs

@@ -410,11 +410,11 @@ pub(crate) fn describe(
     let now = time_of_day()?;
     let sys_v_ipc = SysVIpc::new(id)?;
 
-    if let Some(id) = id {
-        if sys_v_ipc.0.len() != 1 {
-            eprintln!("id {id} not found");
-            return Ok(());
-        }
+    if let Some(id) = id
+        && sys_v_ipc.0.len() != 1
+    {
+        eprintln!("id {id} not found");
+        return Ok(());
     }
 
     table.set_name(c"semaphores")?;

src/uu/lsipc/src/shared_memory.rs

@@ -391,11 +391,11 @@ pub(crate) fn describe(
     let now = time_of_day()?;
     let sys_v_ipc = SysVIpc::new(id)?;
 
-    if let Some(id) = id {
-        if sys_v_ipc.0.len() != 1 {
-            eprintln!("id {id} not found");
-            return Ok(());
-        }
+    if let Some(id) = id
+        && sys_v_ipc.0.len() != 1
+    {
+        eprintln!("id {id} not found");
+        return Ok(());
     }
 
     table.set_name(c"sharedmemory")?;

src/uu/lslocks/src/utils.rs

@@ -275,22 +275,23 @@ impl LockInfo {
 
         let range = start..end;
 
-        if let Some(pid_locks) = pid_locks {
-            if command_name.is_none() && !blocked {
-                let lock_compare = |lock: &&LockInfo| {
-                    lock.range == range
-                        && lock.inode == inode
-                        && lock.device_id == device_id
-                        && lock.mandatory == mandatory
-                        && lock.blocked == blocked
-                        && lock.kind == kind
-                        && lock.mode == mode
-                };
-
-                if let Some(found) = pid_locks.iter().find(lock_compare) {
-                    process_id = found.process_id;
-                    command_name = found.command_name.clone();
-                }
+        if let Some(pid_locks) = pid_locks
+            && command_name.is_none()
+            && !blocked
+        {
+            let lock_compare = |lock: &&LockInfo| {
+                lock.range == range
+                    && lock.inode == inode
+                    && lock.device_id == device_id
+                    && lock.mandatory == mandatory
+                    && lock.blocked == blocked
+                    && lock.kind == kind
+                    && lock.mode == mode
+            };
+
+            if let Some(found) = pid_locks.iter().find(lock_compare) {
+                process_id = found.process_id;
+                command_name = found.command_name.clone();
             }
         }