From 2df6b2879ad134a56b8486b7224f866c689876cd Mon Sep 17 00:00:00 2001 From: Matheus Afonso Martins Moreira Date: Sun, 25 Feb 2024 14:10:04 -0300 Subject: [PATCH] faces/clock: refactor time signal toggling code Simplifies the code by defining dedicated functions for this. --- movement/watch_faces/clock/clock_face.c | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/movement/watch_faces/clock/clock_face.c b/movement/watch_faces/clock/clock_face.c index 7e4ec816..f6c3f3f4 100644 --- a/movement/watch_faces/clock/clock_face.c +++ b/movement/watch_faces/clock/clock_face.c @@ -101,6 +101,11 @@ static void clock_indicate_low_available_power(clock_state_t *clock) { clock_indicate(WATCH_INDICATOR_LAP, clock->battery_low); } +static void clock_toggle_time_signal(clock_state_t *clock) { + clock->time_signal_enabled = !clock->time_signal_enabled; + clock_indicate_time_signal(clock); +} + static void clock_display_all(watch_date_time date_time) { char buf[10 + 1]; @@ -240,8 +245,7 @@ bool clock_face_loop(movement_event_t event, movement_settings_t *settings, void break; case EVENT_ALARM_LONG_PRESS: - state->time_signal_enabled = !state->time_signal_enabled; - clock_indicate_time_signal(state); + clock_toggle_time_signal(state); break; case EVENT_BACKGROUND_TASK: // uncomment this line to snap back to the clock face when the hour signal sounds: