anas-elgarhy/cmus-notify
1
0
Fork 0
Code Issues 1 Pull requests 1 Projects Releases 1 Packages Wiki Activity

Update code to replace manual action assignment with direct return from event.build_notification()

Browse source
This commit is contained in:
Anas Elgarhy 2023-02-21 04:47:05 +02:00
parent 607d53a4a0
commit e34d676eb9
No known key found for this signature in database
GPG key ID: 0501802A1D496528
1 changed files with 1 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/notification.rs
View file

@ -53,13 +53,11 @@ impl NotificationsHandler {
//FIXME: Should check if the user has enabled the cover feature or use a static cover. //FIXME: Should check if the user has enabled the cover feature or use a static cover.
self.update_cover(&events[0], response); self.update_cover(&events[0], response);
let mut action = Action::None;
for event in events { for event in events {
#[cfg(feature = "debug")] #[cfg(feature = "debug")]
info!("event: {:?}", event); info!("event: {:?}", event);
action = event.build_notification(&mut self.settings, &mut self.notification); let action = event.build_notification(&mut self.settings, &mut self.notification);
match action { match action {
Action::Show => { Action::Show => {