Merge pull request #1379 from PolyMeilex/fix/clippy

Address Clippy lints

4 files changed, 24 insertions, 14 deletions

diff --git a/futures/src/lib.rs b/futures/src/lib.rs
index b0b2f6ce..c0982db7 100644
--- a/futures/src/lib.rs
+++ b/futures/src/lib.rs
@@ -4,13 +4,19 @@
 #![doc(
     html_logo_url = "https://raw.githubusercontent.com/iced-rs/iced/9ab6923e943f784985e9ef9ca28b10278297225d/docs/logo.svg"
 )]
-#![deny(missing_docs)]
-#![deny(missing_debug_implementations)]
-#![deny(unused_results)]
-#![forbid(unsafe_code)]
-#![forbid(rust_2018_idioms)]
+#![deny(
+    missing_debug_implementations,
+    missing_docs,
+    unused_results,
+    clippy::extra_unused_lifetimes,
+    clippy::from_over_into,
+    clippy::needless_borrow,
+    clippy::new_without_default,
+    clippy::useless_conversion
+)]
+#![forbid(unsafe_code, rust_2018_idioms)]
+#![allow(clippy::inherent_to_string, clippy::type_complexity)]
 #![cfg_attr(docsrs, feature(doc_cfg))]
-
 pub use futures;
 
 mod command;
diff --git a/futures/src/runtime.rs b/futures/src/runtime.rs
index 34f6b6dd..24f9f241 100644
--- a/futures/src/runtime.rs
+++ b/futures/src/runtime.rs
@@ -66,8 +66,6 @@ where
 
         let future = future.then(|message| async move {
             let _ = sender.send(message).await;
-
-            ()
         });
 
         self.executor.spawn(future);
diff --git a/futures/src/subscription.rs b/futures/src/subscription.rs
index 0085886d..0479c63c 100644
--- a/futures/src/subscription.rs
+++ b/futures/src/subscription.rs
@@ -183,11 +183,7 @@ where
 
         let mapper = self.mapper;
 
-        Box::pin(
-            self.recipe
-                .stream(input)
-                .map(move |element| mapper(element)),
-        )
+        Box::pin(self.recipe.stream(input).map(mapper))
     }
 }
 
diff --git a/futures/src/subscription/tracker.rs b/futures/src/subscription/tracker.rs
index 2cf98284..9fe110b0 100644
--- a/futures/src/subscription/tracker.rs
+++ b/futures/src/subscription/tracker.rs
@@ -127,7 +127,7 @@ where
             futures.push(Box::pin(future));
         }
 
-        self.subscriptions.retain(|id, _| alive.contains(&id));
+        self.subscriptions.retain(|id, _| alive.contains(id));
 
         futures
     }
@@ -156,3 +156,13 @@ where
             });
     }
 }
+
+impl<Hasher, Event> Default for Tracker<Hasher, Event>
+where
+    Hasher: std::hash::Hasher + Default,
+    Event: 'static + Send + Clone,
+{
+    fn default() -> Self {
+        Self::new()
+    }
+}