Refactor: Don't clone references

This commit is contained in:
Henrik Friedrichsen
2022-12-15 23:42:55 +01:00
parent 23e9ecfd5c
commit 703ffc76ba

View File

@@ -295,13 +295,13 @@ impl WebApi {
let seed_artistids = seed_artists.as_ref().map(|artistids| { let seed_artistids = seed_artists.as_ref().map(|artistids| {
artistids artistids
.iter() .iter()
.map(|id| ArtistId::from_id(id.clone()).unwrap()) .map(|id| ArtistId::from_id(*id).unwrap())
.collect::<Vec<ArtistId>>() .collect::<Vec<ArtistId>>()
}); });
let seed_trackids = seed_tracks.as_ref().map(|trackids| { let seed_trackids = seed_tracks.as_ref().map(|trackids| {
trackids trackids
.iter() .iter()
.map(|id| TrackId::from_id(id.clone()).unwrap()) .map(|id| TrackId::from_id(*id).unwrap())
.collect::<Vec<TrackId>>() .collect::<Vec<TrackId>>()
}); });
api.recommendations( api.recommendations(
@@ -485,7 +485,7 @@ impl WebApi {
self.api_with_retry(|api| { self.api_with_retry(|api| {
api.save_shows( api.save_shows(
ids.iter() ids.iter()
.map(|id| ShowId::from_id(id.clone()).unwrap()) .map(|id| ShowId::from_id(*id).unwrap())
.collect::<Vec<ShowId>>(), .collect::<Vec<ShowId>>(),
) )
}) })
@@ -496,7 +496,7 @@ impl WebApi {
self.api_with_retry(|api| { self.api_with_retry(|api| {
api.remove_users_saved_shows( api.remove_users_saved_shows(
ids.iter() ids.iter()
.map(|id| ShowId::from_id(id.clone()).unwrap()) .map(|id| ShowId::from_id(*id).unwrap())
.collect::<Vec<ShowId>>(), .collect::<Vec<ShowId>>(),
Some(Market::FromToken), Some(Market::FromToken),
) )
@@ -515,7 +515,7 @@ impl WebApi {
self.api_with_retry(|api| { self.api_with_retry(|api| {
api.user_follow_artists( api.user_follow_artists(
ids.iter() ids.iter()
.map(|id| ArtistId::from_id(id.clone()).unwrap()) .map(|id| ArtistId::from_id(*id).unwrap())
.collect::<Vec<ArtistId>>(), .collect::<Vec<ArtistId>>(),
) )
}) })
@@ -525,7 +525,7 @@ impl WebApi {
self.api_with_retry(|api| { self.api_with_retry(|api| {
api.user_unfollow_artists( api.user_unfollow_artists(
ids.iter() ids.iter()
.map(|id| ArtistId::from_id(id.clone()).unwrap()) .map(|id| ArtistId::from_id(*id).unwrap())
.collect::<Vec<ArtistId>>(), .collect::<Vec<ArtistId>>(),
) )
}) })
@@ -541,7 +541,7 @@ impl WebApi {
self.api_with_retry(|api| { self.api_with_retry(|api| {
api.current_user_saved_albums_add( api.current_user_saved_albums_add(
ids.iter() ids.iter()
.map(|id| AlbumId::from_id(id.clone()).unwrap()) .map(|id| AlbumId::from_id(*id).unwrap())
.collect::<Vec<AlbumId>>(), .collect::<Vec<AlbumId>>(),
) )
}) })
@@ -551,7 +551,7 @@ impl WebApi {
self.api_with_retry(|api| { self.api_with_retry(|api| {
api.current_user_saved_albums_delete( api.current_user_saved_albums_delete(
ids.iter() ids.iter()
.map(|id| AlbumId::from_id(id.clone()).unwrap()) .map(|id| AlbumId::from_id(*id).unwrap())
.collect::<Vec<AlbumId>>(), .collect::<Vec<AlbumId>>(),
) )
}) })
@@ -567,7 +567,7 @@ impl WebApi {
self.api_with_retry(|api| { self.api_with_retry(|api| {
api.current_user_saved_tracks_add( api.current_user_saved_tracks_add(
ids.iter() ids.iter()
.map(|id| TrackId::from_id(id.clone()).unwrap()) .map(|id| TrackId::from_id(*id).unwrap())
.collect::<Vec<TrackId>>(), .collect::<Vec<TrackId>>(),
) )
}) })
@@ -577,7 +577,7 @@ impl WebApi {
self.api_with_retry(|api| { self.api_with_retry(|api| {
api.current_user_saved_tracks_delete( api.current_user_saved_tracks_delete(
ids.iter() ids.iter()
.map(|id| TrackId::from_id(id.clone()).unwrap()) .map(|id| TrackId::from_id(*id).unwrap())
.collect::<Vec<TrackId>>(), .collect::<Vec<TrackId>>(),
) )
}) })