From 5a682e28895d7b1122f71b9d331114573985e08b Mon Sep 17 00:00:00 2001 From: Rudxain <76864299+Rudxain@users.noreply.github.com> Date: Wed, 5 Mar 2025 23:10:20 -0400 Subject: [PATCH] refactor: `cargo clippy --fix` --- src/core/save.rs | 8 ++++---- src/core/update.rs | 8 ++++---- src/gui/views/settings.rs | 6 +++--- 3 files changed, 11 insertions(+), 11 deletions(-) diff --git a/src/core/save.rs b/src/core/save.rs index ee454a8..79b7e97 100644 --- a/src/core/save.rs +++ b/src/core/save.rs @@ -138,19 +138,19 @@ pub fn restore_backup( }; for (i, backup_package) in u.packages.iter().enumerate() { - let package: CorePackage; - match packages[index] + + let package: CorePackage = match packages[index] .iter() .find(|x| x.name == backup_package.name) { - Some(p) => package = p.into(), + Some(p) => p.into(), None => { return Err(format!( "{} not found for user {}", backup_package.name, u.id )); } - } + }; let p_commands = apply_pkg_state_commands( &package, backup_package.state, diff --git a/src/core/update.rs b/src/core/update.rs index d2029f4..69081b1 100644 --- a/src/core/update.rs +++ b/src/core/update.rs @@ -182,8 +182,8 @@ pub fn get_latest_release() -> Result, ()> { pub fn get_latest_release() -> Result, ()> { debug!("Checking for {NAME} update"); - match ureq::get("https://api.github.com/repos/Universal-Debloater-Alliance/universal-android-debloater/releases/latest") - .call() { Ok(res) => { + if let Ok(res) = ureq::get("https://api.github.com/repos/Universal-Debloater-Alliance/universal-android-debloater/releases/latest") + .call() { let release: Release = serde_json::from_value( res.into_json::() .map_err(|_| ())? @@ -200,10 +200,10 @@ pub fn get_latest_release() -> Result, ()> { } else { Ok(None) } - } _ => { + } else { debug!("Failed to check {NAME} update"); Err(()) - }} + } } /// Extracts the binary from a `tar.gz` archive to `temp_file` path diff --git a/src/gui/views/settings.rs b/src/gui/views/settings.rs index 7a4bfb9..dfcd6f6 100644 --- a/src/gui/views/settings.rs +++ b/src/gui/views/settings.rs @@ -465,7 +465,9 @@ impl Settings { .spacing(10) .align_items(Alignment::Center); - let restore_row = if !self.device.backup.backups.is_empty() { + let restore_row = if self.device.backup.backups.is_empty() { + row![] + } else { row![ restore_btn(true), "Restore the state of the device", @@ -475,8 +477,6 @@ impl Settings { ] .spacing(10) .align_items(Alignment::Center) - } else { - row![] }; let no_device_ctn = || {