From 8d1c17ba7dcb2da08379cbcfcbfdf2d20b906849 Mon Sep 17 00:00:00 2001
From: Evan Almloff <evanalmloff@gmail.com>
Date: Wed, 9 Aug 2023 11:10:41 -0700
Subject: [PATCH] fix clippy

---
 packages/cli/src/config.rs           |  2 +-
 packages/core/tests/suspense.rs      |  2 +-
 packages/desktop/src/protocol.rs     |  9 +++++++--
 packages/html/src/eval.rs            |  2 +-
 packages/native-core/src/real_dom.rs |  1 -
 packages/rsx/src/lib.rs              | 10 ++--------
 6 files changed, 12 insertions(+), 14 deletions(-)

diff --git a/packages/cli/src/config.rs b/packages/cli/src/config.rs
index 2fc9f96ce..267a7f2a2 100644
--- a/packages/cli/src/config.rs
+++ b/packages/cli/src/config.rs
@@ -126,7 +126,7 @@ impl Default for DioxusConfig {
             },
             bundle: BundleConfig {
                 identifier: Some(format!("io.github.{name}")),
-                publisher: Some(name.into()),
+                publisher: Some(name),
                 ..Default::default()
             },
             plugin: toml::Value::Table(toml::map::Map::new()),
diff --git a/packages/core/tests/suspense.rs b/packages/core/tests/suspense.rs
index 7b2516447..abce7f8a6 100644
--- a/packages/core/tests/suspense.rs
+++ b/packages/core/tests/suspense.rs
@@ -36,7 +36,7 @@ fn suspended_child(cx: Scope) -> Element {
         cx.spawn(async move {
             val += 1;
         });
-        return cx.suspend()?;
+        cx.suspend()?;
     }
 
     render!("child")
diff --git a/packages/desktop/src/protocol.rs b/packages/desktop/src/protocol.rs
index 89ce50a48..53652f82f 100644
--- a/packages/desktop/src/protocol.rs
+++ b/packages/desktop/src/protocol.rs
@@ -158,8 +158,13 @@ fn get_mime_from_path(trimmed: &Path) -> Result<&'static str> {
     }
 
     let res = match infer::get_from_path(trimmed)?.map(|f| f.mime_type()) {
-        Some(t) if t == "text/plain" => get_mime_by_ext(trimmed),
-        Some(f) => f,
+        Some(f) => {
+            if f == "text/plain" {
+                get_mime_by_ext(trimmed)
+            } else {
+                f
+            }
+        }
         None => get_mime_by_ext(trimmed),
     };
 
diff --git a/packages/html/src/eval.rs b/packages/html/src/eval.rs
index b39eef805..c90f5125b 100644
--- a/packages/html/src/eval.rs
+++ b/packages/html/src/eval.rs
@@ -42,7 +42,7 @@ pub fn use_eval(cx: &ScopeState) -> &EvalCreator {
         Rc::new(move |script: &str| {
             eval_provider
                 .new_evaluator(script.to_string())
-                .map(|evaluator| UseEval::new(evaluator))
+                .map(UseEval::new)
         }) as Rc<dyn Fn(&str) -> Result<UseEval, EvalError>>
     })
 }
diff --git a/packages/native-core/src/real_dom.rs b/packages/native-core/src/real_dom.rs
index 44c378ddf..b520b2de1 100644
--- a/packages/native-core/src/real_dom.rs
+++ b/packages/native-core/src/real_dom.rs
@@ -446,7 +446,6 @@ impl<V: FromAnyValue + Send + Sync> RealDom<V> {
             drop(tree);
             children.reverse();
             if let Some(node) = self.get_mut(id) {
-                let node = node;
                 f(node);
                 stack.extend(children.iter());
             }
diff --git a/packages/rsx/src/lib.rs b/packages/rsx/src/lib.rs
index 1f01fcd72..783dd3322 100644
--- a/packages/rsx/src/lib.rs
+++ b/packages/rsx/src/lib.rs
@@ -288,10 +288,7 @@ impl DynamicMapping {
         let idx = self.last_attribute_idx;
         self.last_attribute_idx += 1;
 
-        self.attribute_to_idx
-            .entry(attr)
-            .or_insert_with(Vec::new)
-            .push(idx);
+        self.attribute_to_idx.entry(attr).or_default().push(idx);
 
         idx
     }
@@ -300,10 +297,7 @@ impl DynamicMapping {
         let idx = self.last_element_idx;
         self.last_element_idx += 1;
 
-        self.node_to_idx
-            .entry(node)
-            .or_insert_with(Vec::new)
-            .push(idx);
+        self.node_to_idx.entry(node).or_default().push(idx);
 
         idx
     }