Fix clippy warnings - Githubissues

thirteenowls commented 3 months ago

Description

Fixed clippy warnings.

Motivation and Context

https://github.com/ramsayleung/rspotify/pull/483#issuecomment-2123875453

Dependencies

None.

Type of change

Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How has this been tested?

By running clippy and observing no warnings.

Is this change properly documented?

N/A

thirteenowls commented 3 months ago

Missed a couple, sorry. I'd only tested using the feature combinations I saw fail in CI. clippy::assigning_clones will be moved to pedantic in the next release, and its current recommendation to use ToOwned::clone_into is incompatible with rspotify's MSRV, so I thought it'd be best to simply allow it for the time being.

ramsayleung commented 3 months ago

No worries, thanks for your contribution.

ramsayleung / rspotify

Fix clippy warnings #484