Improved api

Author: Johannesd3

discovery/examples/discovery.rs

@@ -10,7 +10,7 @@ async fn main() {
         .init()
         .unwrap();
 
-    let name = "Librespot".to_string();
+    let name = "Librespot";
     let device_id = hex::encode(Sha1::digest(name.as_bytes()));
 
     let mut server = librespot_discovery::Discovery::builder(device_id)

discovery/src/lib.rs

@@ -75,18 +75,18 @@ pub enum Error {
 
 impl Builder {
     /// Starts a new builder using the provided device id.
-    pub fn new(device_id: String) -> Self {
+    pub fn new(device_id: impl Into<String>) -> Self {
         Self {
             name: "Librespot".into(),
             device_type: DeviceType::default(),
-            device_id,
+            device_id: device_id.into(),
             port: 0,
         }
     }
 
     /// Sets the name to be displayed. Default is `"Librespot"`.
-    pub fn name(mut self, name: String) -> Self {
-        self.name = name;
+    pub fn name(mut self, name: impl Into<String>) -> Self {
+        self.name = name.into();
         self
     }
 
@@ -143,12 +143,12 @@ impl Builder {
 
 impl Discovery {
     /// Starts a [`Builder`] with the provided device id.
-    pub fn builder(device_id: String) -> Builder {
+    pub fn builder(device_id: impl Into<String>) -> Builder {
         Builder::new(device_id)
     }
 
     /// Create a new instance with the specified device id and default paramaters.
-    pub fn new(device_id: String) -> Result<Self, Error> {
+    pub fn new(device_id: impl Into<String>) -> Result<Self, Error> {
         Self::builder(device_id).launch()
     }
 }