Fix: applied clippy suggestions + format

src/mpris.rs

@@ -465,11 +465,12 @@ impl MprisPlayer {
 /// Commands to control the [MprisManager] worker thread.
 #[derive(Debug)]
 pub enum MprisCommand {
-    /// Notify about playback status and metadata updates.
-    PlaybackUpdate,
-    /// Notify about volume updates.
-    VolumeUpdate,
-    MetadataUpdate,
+    /// Emit playback status
+    Playback,
+    /// Emit volume
+    Volume,
+    /// Emit metadata
+    Metadata,
 }
 
 /// An MPRIS server that internally manager a thread which can be sent commands. This is internally
@@ -527,14 +528,14 @@ impl MprisManager {
         loop {
             let ctx = player_iface_ref.signal_context();
             match rx.next().await {
-                Some(MprisCommand::PlaybackUpdate) => {
+                Some(MprisCommand::Playback) => {
                     player_iface.playback_status_changed(ctx).await?;
                 }
-                Some(MprisCommand::VolumeUpdate) => {
+                Some(MprisCommand::Volume) => {
                     info!("sending MPRIS volume update signal");
                     player_iface.volume_changed(ctx).await?;
                 }
-                Some(MprisCommand::MetadataUpdate) => {
+                Some(MprisCommand::Metadata) => {
                     player_iface.metadata_changed(ctx).await?;
                 }
                 None => break,

src/queue.rs

@@ -8,7 +8,7 @@ use notify_rust::Notification;
 use rand::prelude::*;
 use strum_macros::Display;
 
-use crate::config::{Config};
+use crate::config::Config;
 use crate::library::Library;
 use crate::model::playable::Playable;
 use crate::spotify::PlayerEvent;

src/spotify.rs

@@ -317,7 +317,7 @@ impl Spotify {
         ));
 
         #[cfg(feature = "mpris")]
-        self.send_mpris(MprisCommand::MetadataUpdate);
+        self.send_mpris(MprisCommand::Metadata);
     }
 
     /// Update the cached status of the [Player]. This makes sure the status
@@ -342,8 +342,8 @@ impl Spotify {
         let mut status = self.status.write().unwrap();
         *status = new_status;
 
-        #[cfg(feature ="mpris")]
-        self.send_mpris(MprisCommand::PlaybackUpdate);
+        #[cfg(feature = "mpris")]
+        self.send_mpris(MprisCommand::Playback);
     }
 
     /// Reset the time tracking stats for the current song. This should be called when a new song is
@@ -369,7 +369,7 @@ impl Spotify {
     }
 
     /// Send an [MprisCommand] to the mpris thread.
-    #[cfg(feature ="mpris")]
+    #[cfg(feature = "mpris")]
     fn send_mpris(&self, cmd: MprisCommand) {
         debug!("Sending mpris command: {:?}", cmd);
         if let Some(mpris_manager) = self.mpris.lock().unwrap().as_ref() {
@@ -435,7 +435,7 @@ impl Spotify {
         // MPRIS implementation.
         if notify {
             #[cfg(feature = "mpris")]
-            self.send_mpris(MprisCommand::VolumeUpdate)
+            self.send_mpris(MprisCommand::Volume)
         }
     }