Merge pull request #3875 from alexlnkp/master

Alexays · web-flow · commit 369c81d6f29e · 2025-01-10T09:34:00.000+01:00
fix: never sleep cava when sleep_timer is 0
diff --git a/src/modules/cava.cpp b/src/modules/cava.cpp
@@ -139,15 +139,15 @@ auto waybar::modules::Cava::update() -> void {
     }
   }
 
-  if (silence_ && prm_.sleep_timer) {
+  if (silence_ && prm_.sleep_timer != 0) {
     if (sleep_counter_ <=
         (int)(std::chrono::milliseconds(prm_.sleep_timer * 1s) / frame_time_milsec_)) {
       ++sleep_counter_;
       silence_ = false;
     }
   }
 
-  if (!silence_) {
+  if (!silence_ || prm_.sleep_timer == 0) {
     downThreadDelay(frame_time_milsec_, suspend_silence_delay_);
     // Process: execute cava
     pthread_mutex_lock(&audio_data_.lock);

Original file line number	Diff line number	Diff line change
`@@ -139,15 +139,15 @@ auto waybar::modules::Cava::update() -> void {`
`139`	`139`	`}`
`140`	`140`	`}`
`141`	`141`
`142`		`- if (silence_ && prm_.sleep_timer) {`
	`142`	`+ if (silence_ && prm_.sleep_timer != 0) {`
`143`	`143`	`if (sleep_counter_ <=`
`144`	`144`	`(int)(std::chrono::milliseconds(prm_.sleep_timer * 1s) / frame_time_milsec_)) {`
`145`	`145`	`++sleep_counter_;`
`146`	`146`	`silence_ = false;`
`147`	`147`	`}`
`148`	`148`	`}`
`149`	`149`
`150`		`- if (!silence_) {`
	`150`	`+ if (!silence_ \|\| prm_.sleep_timer == 0) {`
`151`	`151`	`downThreadDelay(frame_time_milsec_, suspend_silence_delay_);`
`152`	`152`	`// Process: execute cava`
`153`	`153`	`pthread_mutex_lock(&audio_data_.lock);`