Fix new clippy lints (#2713)

src/client/mod.rs

@@ -316,8 +316,9 @@ impl IntoFuture for ClientBuilder {
         if let Some(ratelimiter) = &mut http.ratelimiter {
             let event_handlers_clone = event_handlers.clone();
             ratelimiter.set_ratelimit_callback(Box::new(move |info| {
-                for event_handler in event_handlers_clone.iter().map(Arc::clone) {
+                for event_handler in &event_handlers_clone {
                     let info = info.clone();
+                    let event_handler = Arc::clone(event_handler);
                     tokio::spawn(async move { event_handler.ratelimit(info).await });
                 }
             }));
@@ -349,7 +350,7 @@ impl IntoFuture for ClientBuilder {
                 #[cfg(feature = "framework")]
                 framework: Arc::clone(&framework_cell),
                 #[cfg(feature = "voice")]
-                voice_manager: voice_manager.as_ref().map(Arc::clone),
+                voice_manager: voice_manager.clone(),
                 ws_url: Arc::clone(&ws_url),
                 shard_total,
                 #[cfg(feature = "cache")]

src/gateway/bridge/shard_queuer.rs

@@ -186,7 +186,7 @@ impl ShardQueuer {
             event_handlers: self.event_handlers.clone(),
             raw_event_handlers: self.raw_event_handlers.clone(),
             #[cfg(feature = "framework")]
-            framework: self.framework.get().map(Arc::clone),
+            framework: self.framework.get().cloned(),
             manager: Arc::clone(&self.manager),
             #[cfg(feature = "voice")]
             voice_manager: self.voice_manager.clone(),

src/model/utils.rs

@@ -339,7 +339,7 @@ pub mod comma_separated_string {
         vec: &FixedArray<FixedString>,
         serializer: S,
     ) -> Result<S::Ok, S::Error> {
-        let vec: Vec<String> = vec.iter().map(FixedString::clone).map(String::from).collect();
+        let vec: Vec<String> = vec.iter().cloned().map(String::from).collect();
         serializer.serialize_str(&vec.join(", "))
     }
 }