🧹 make clippy happy

CachyOS · Sep 23, 2023 · 33113f0 · 33113f0
1 parent ccc14a5
commit 33113f0
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 4 deletions.
diff --git a/src/data.rs b/src/data.rs
@@ -58,7 +58,7 @@ impl Data {
         // Refill data
         self.fill_installed_profiles();
 
-        set_matching_profiles(&mut self.pci_devices, &mut self.installed_pci_profiles, true);
+        set_matching_profiles(&mut self.pci_devices, &self.installed_pci_profiles, true);
     }
 
     fn fill_installed_profiles(&mut self) {
@@ -84,7 +84,7 @@ impl Data {
 
         self.fill_all_profiles();
 
-        set_matching_profiles(&mut self.pci_devices, &mut self.all_pci_profiles, false);
+        set_matching_profiles(&mut self.pci_devices, &self.all_pci_profiles, false);
 
         self.update_installed_profile_data();
     }

diff --git a/src/profile.rs b/src/profile.rs
@@ -211,12 +211,16 @@ fn parse_profile(node: &toml::Table, profile_name: &str) -> Result<Profile> {
 }
 
 fn parse_ids_file(file_path: &str) -> Result<String> {
+    use std::fmt::Write;
+
     let file_content = fs::read_to_string(file_path)?;
     let parsed_ids = file_content
         .lines()
         .filter(|x| !x.trim().is_empty() && x.trim().as_bytes()[0] != b'#')
-        .map(|x| format!(" {}", x.trim()))
-        .collect::<String>();
+        .fold(String::new(), |mut output, x| {
+            let _ = write!(output, " {}", x.trim());
+            output
+        });
 
     Ok(parsed_ids.split_ascii_whitespace().collect::<Vec<_>>().join(" "))
 }