move buzzer enabled detection logic into movement for movement_play_signal/tune

this way watch faces don't have to disable/enable the buzzer themselves
before calling movement_play_signal() and movement_play_tune()

diff --git a/movement/movement.c b/movement/movement.c
index f2292e5f4cf5f0a5b61eb8df53e0726ad20e510e..f0868416eb0b2f9cee5271a732692f8c0e5aea91 100644 (file)
--- a/movement/movement.c
+++ b/movement/movement.c
@@ -294,25 +294,25 @@ void movement_request_wake() {
 }
 
 void movement_play_signal(void) {
+    bool buzzer_enabled = watch_is_buzzer_or_led_enabled();
+    if (!buzzer_enabled) {
+        watch_enable_buzzer();
+    }
     watch_buzzer_play_note(BUZZER_NOTE_C8, 75);
     watch_buzzer_play_note(BUZZER_NOTE_REST, 100);
     watch_buzzer_play_note(BUZZER_NOTE_C8, 100);
-}
-
-void movement_play_signal_background(void) {
-    watch_enable_buzzer();
-    movement_play_signal();
-    watch_disable_buzzer();
+    if (!buzzer_enabled) {
+        watch_disable_buzzer();
+    }
 }
 
 void movement_play_tune(void) {
-    watch_buzzer_play_sequence(signal_tune, NULL);
-}
-
-/* Special version of movement_play_tune meant for use during LE mode */
-void movement_play_tune_background(void) {
-    watch_enable_buzzer();
-    watch_buzzer_play_sequence(signal_tune, watch_disable_buzzer);
+    if (!watch_is_buzzer_or_led_enabled()) {
+        watch_enable_buzzer();
+        watch_buzzer_play_sequence(signal_tune, watch_disable_buzzer);
+    } else {
+        watch_buzzer_play_sequence(signal_tune, NULL);
+    }
 }
 
 void movement_play_alarm(void) {

diff --git a/movement/movement.h b/movement/movement.h
index c6380cec74ecdaf41c304263cae9bdf3cb1787c3..5f30dfb8c652413808665e7870f54013a6316e7d 100644 (file)
--- a/movement/movement.h
+++ b/movement/movement.h
@@ -307,9 +307,7 @@ void movement_cancel_background_task_for_face(uint8_t watch_face_index);
 void movement_request_wake(void);
 
 void movement_play_signal(void);
-void movement_play_signal_background(void);
 void movement_play_tune(void);
-void movement_play_tune_background(void);
 void movement_play_alarm(void);
 void movement_play_alarm_beeps(uint8_t rounds, BuzzerNote alarm_note);
 

diff --git a/movement/watch_faces/clock/repetition_minute_face.c b/movement/watch_faces/clock/repetition_minute_face.c
index 917c57150f31f922974b04087042ebf2983fa250..fc78b2d860b67dcdc11eec1f0ed7c07e395490f2 100644 (file)
--- a/movement/watch_faces/clock/repetition_minute_face.c
+++ b/movement/watch_faces/clock/repetition_minute_face.c
@@ -151,11 +151,7 @@ bool repetition_minute_face_loop(movement_event_t event, movement_settings_t *se
             else watch_clear_indicator(WATCH_INDICATOR_BELL);
             break;
         case EVENT_BACKGROUND_TASK:
-            if (watch_is_buzzer_or_led_enabled()) {
-                movement_play_signal();
-            } else {
-                movement_play_signal_background();
-            }
+            movement_play_signal();
             break;
         case EVENT_LIGHT_LONG_UP:
             /*

diff --git a/movement/watch_faces/clock/simple_clock_bin_led_face.c b/movement/watch_faces/clock/simple_clock_bin_led_face.c
index 09c8457b9e40cb0457456fa55b6327decc5d832f..cf39c1886bb8aa8908a2a47b91a494f9a967e2bf 100644 (file)
--- a/movement/watch_faces/clock/simple_clock_bin_led_face.c
+++ b/movement/watch_faces/clock/simple_clock_bin_led_face.c
@@ -180,11 +180,7 @@ bool simple_clock_bin_led_face_loop(movement_event_t event, movement_settings_t
         case EVENT_BACKGROUND_TASK:
             // uncomment this line to snap back to the clock face when the hour signal sounds:
             // movement_move_to_face(state->watch_face_index);
-            if (watch_is_buzzer_or_led_enabled()) {
-                movement_play_signal();
-            } else {
-                movement_play_signal_background();
-            }
+            movement_play_signal();
             break;
         case EVENT_LIGHT_LONG_PRESS:
             if (state->flashing_state == 0) {

diff --git a/movement/watch_faces/clock/simple_clock_face.c b/movement/watch_faces/clock/simple_clock_face.c
index ebe7542f0d3772ffc1eb3836d9706151b576e651..63f49234570f722594eea2db823a37e6d978007b 100644 (file)
--- a/movement/watch_faces/clock/simple_clock_face.c
+++ b/movement/watch_faces/clock/simple_clock_face.c
@@ -136,11 +136,7 @@ bool simple_clock_face_loop(movement_event_t event, movement_settings_t *setting
         case EVENT_BACKGROUND_TASK:
             // uncomment this line to snap back to the clock face when the hour signal sounds:
             // movement_move_to_face(state->watch_face_index);
-            if (watch_is_buzzer_or_led_enabled()) {
-                movement_play_tune();
-            } else {
-                movement_play_tune_background();
-            }
+            movement_play_tune();
             break;
         default:
             return movement_default_loop_handler(event, settings);

diff --git a/movement/watch_faces/clock/weeknumber_clock_face.c b/movement/watch_faces/clock/weeknumber_clock_face.c
index b4514bf143bdfc060aa33f68d797c6b4b53d5aa9..81df58475ba962dcea335112280e891d76927f7e 100644 (file)
--- a/movement/watch_faces/clock/weeknumber_clock_face.c
+++ b/movement/watch_faces/clock/weeknumber_clock_face.c
@@ -130,11 +130,7 @@ bool weeknumber_clock_face_loop(movement_event_t event, movement_settings_t *set
         case EVENT_BACKGROUND_TASK:
             // uncomment this line to snap back to the clock face when the hour signal sounds:
             // movement_move_to_face(state->watch_face_index);
-            if (watch_is_buzzer_or_led_enabled()) {
-                movement_play_signal();
-            } else {
-                movement_play_signal_background();
-            }
+            movement_play_signal();
             break;
         default:
             movement_default_loop_handler(event, settings);