OscMessage::new takes `&impl ToString`, should take `impl ToString`

benjhar / osc

Open Sound Control library for Rust

Apache License 2.0

0 stars 0 forks source link

OscMessage::new takes `&impl ToString`, should take `impl ToString` #2

Closed benjhar closed 2 months ago

benjhar commented 2 months ago

While clippy suggests taking a &impl ToString, we do want to consume the value, and requiring a '&' on static strings feels very odd and gives a confusing error message.

benjhar commented 2 months ago

749ebe8 fixed this issue.