clippy: enable implicit_clone lint

https://rust-lang.github.io/rust-clippy/master/index.html#implicit_clone

Author: xtqqczze

Cargo.toml

@@ -211,6 +211,7 @@ assets = [
 ]
 
 [workspace.lints]
+clippy.implicit_clone = "warn"
 clippy.redundant_closure_for_method_calls = "warn"
 
 [lints]

connect/src/state/context.rs

@@ -468,14 +468,14 @@ impl ConnectState {
         //  - queue resorting is done by each client and orients itself by the given uid
         //  - if no uid is present, resorting doesn't work or behaves not as intended
         let uid = match ctx_track.uid.as_ref() {
-            Some(uid) if !uid.is_empty() => uid.to_string(),
+            Some(uid) if !uid.is_empty() => uid.clone(),
             // so providing a unique id should allow to resort the queue
             _ => Uuid::new_v4().as_simple().to_string(),
         };
 
         let mut metadata = page_metadata.cloned().unwrap_or_default();
         for (k, v) in &ctx_track.metadata {
-            metadata.insert(k.to_string(), v.to_string());
+            metadata.insert(k.clone(), v.clone());
         }
 
         let mut track = ProvidedTrack {

core/src/login5.rs

@@ -183,7 +183,7 @@ impl Login5Manager {
         }
 
         let method = Login_method::StoredCredential(StoredCredential {
-            username: self.session().username().to_string(),
+            username: self.session().username().clone(),
             data: auth_data,
             ..Default::default()
         });

core/src/spclient.rs

@@ -306,7 +306,7 @@ impl SpClient {
                                 challenge_answer.ChallengeType =
                                     ChallengeType::CHALLENGE_HASH_CASH.into();
 
-                                challenge_answers.state = state.to_string();
+                                challenge_answers.state = state.clone();
                                 challenge_answers.answers.push(challenge_answer);
 
                                 trace!("Answering hash cash challenge");
@@ -347,7 +347,7 @@ impl SpClient {
         };
 
         let granted_token = token_response.granted_token();
-        let access_token = granted_token.token.to_owned();
+        let access_token = granted_token.token.clone();
 
         self.lock(|inner| {
             let client_token = Token {

core/src/token.rs

@@ -97,7 +97,7 @@ impl TokenProvider {
         );
         let request = self.session().mercury().get(query_uri)?;
         let response = request.await?;
-        let data = response.payload.first().ok_or(TokenError::Empty)?.to_vec();
+        let data = response.payload.first().ok_or(TokenError::Empty)?.clone();
         let token = Token::from_json(String::from_utf8(data)?)?;
         trace!("Got token: {token:#?}");
         self.lock(|inner| inner.tokens.push(token.clone()));

discovery/src/server.rs

@@ -156,7 +156,7 @@ impl RequestHandler {
 
         let encrypted_blob_len = encrypted_blob.len();
         if encrypted_blob_len < 16 {
-            return Err(DiscoveryError::HmacError(encrypted_blob.to_vec()).into());
+            return Err(DiscoveryError::HmacError(encrypted_blob.clone()).into());
         }
 
         let iv = &encrypted_blob[0..16];

metadata/src/album.rs

@@ -101,7 +101,7 @@ impl TryFrom<&<Self as Metadata>::Message> for Album {
             covers: album.cover_group.get_or_default().into(),
             external_ids: album.external_id.as_slice().into(),
             discs: album.disc.as_slice().try_into()?,
-            reviews: album.review.to_vec(),
+            reviews: album.review.clone(),
             copyrights: album.copyright.as_slice().into(),
             restrictions: album.restriction.as_slice().into(),
             related: album.related.as_slice().try_into()?,

metadata/src/availability.rs

@@ -41,7 +41,7 @@ impl TryFrom<&AvailabilityMessage> for Availability {
     type Error = librespot_core::Error;
     fn try_from(availability: &AvailabilityMessage) -> Result<Self, Self::Error> {
         Ok(Self {
-            catalogue_strs: availability.catalogue_str.to_vec(),
+            catalogue_strs: availability.catalogue_str.clone(),
             start: availability.start.get_or_default().try_into()?,
         })
     }

metadata/src/content_rating.rs

@@ -23,7 +23,7 @@ impl From<&ContentRatingMessage> for ContentRating {
     fn from(content_rating: &ContentRatingMessage) -> Self {
         Self {
             country: content_rating.country().to_owned(),
-            tags: content_rating.tag.to_vec(),
+            tags: content_rating.tag.clone(),
         }
     }
 }

metadata/src/episode.rs

@@ -90,7 +90,7 @@ impl TryFrom<&<Self as Metadata>::Message> for Episode {
             audio_previews: episode.audio_preview.as_slice().into(),
             restrictions: episode.restriction.as_slice().into(),
             freeze_frames: episode.freeze_frame.image.as_slice().into(),
-            keywords: episode.keyword.to_vec(),
+            keywords: episode.keyword.clone(),
             allow_background_playback: episode.allow_background_playback(),
             availability: episode.availability.as_slice().try_into()?,
             external_url: episode.external_url().to_owned(),