refactor: 🚨 Fix some lints

Vixea · Oct 14, 2023 · e96b5fe · e96b5fe
1 parent 15cf60a
commit e96b5fe
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 6 deletions.
diff --git a/alvr/client_core/build.rs b/alvr/client_core/build.rs
@@ -14,12 +14,12 @@ fn main() {
 
         let source_files_paths = cpp_paths
             .iter()
-            .filter_map(|path| {
+            .filter(|&path| {
                 path.extension()
                     .filter(|ext| ext.to_string_lossy() == "cpp")
                     .is_some()
-                    .then(|| path.clone())
             })
+            .cloned()
             .collect::<Vec<_>>();
 
         cc::Build::new()

diff --git a/alvr/server/src/c_api.rs b/alvr/server/src/c_api.rs
@@ -1,3 +1,5 @@
+#![allow(dead_code, unused_variables)]
+
 use ash::vk;
 use std::{
     ffi::{c_char, CStr},

diff --git a/alvr/server/src/lib.rs b/alvr/server/src/lib.rs
@@ -147,13 +147,13 @@ pub extern "C" fn shutdown_driver() {
         let hostnames = data_manager_lock
             .client_list()
             .iter()
-            .filter_map(|(hostname, info)| {
-                (!matches!(
+            .filter(|&(_, info)| {
+                !matches!(
                     info.connection_state,
                     ConnectionState::Disconnected | ConnectionState::Disconnecting { .. }
-                ))
-                .then(|| hostname.clone())
+                )
             })
+            .map(|(hostname, _)| hostname.clone())
             .collect::<Vec<_>>();
 
         for hostname in hostnames {