From 4d45fcde304a924a2aef0d29d62d0dece2fcd442 Mon Sep 17 00:00:00 2001 From: Daniel Bergman Date: Tue, 8 Jul 2025 01:14:07 +0200 Subject: [PATCH] Tally face: fix build warnings (#43) Clamp tally index values in print_tally function to ensure they remain within defined limits, silencing compiler warnings --- watch-faces/complication/tally_face.c | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/watch-faces/complication/tally_face.c b/watch-faces/complication/tally_face.c index c2d202e6..deb5b878 100644 --- a/watch-faces/complication/tally_face.c +++ b/watch-faces/complication/tally_face.c @@ -195,12 +195,18 @@ bool tally_face_loop(movement_event_t event, void *context) { // print tally index at the center of display. void print_tally(tally_state_t *state, bool sound_on) { char buf[6]; + int display_val = (int)(state->tally_idx); + + // Clamp to limits + if (display_val > TALLY_FACE_MAX) display_val = TALLY_FACE_MAX; + if (display_val < TALLY_FACE_MIN) display_val = TALLY_FACE_MIN; + if (sound_on) watch_set_indicator(WATCH_INDICATOR_BELL); else watch_clear_indicator(WATCH_INDICATOR_BELL); watch_display_text_with_fallback(WATCH_POSITION_TOP, "TALLY", "TA"); - sprintf(buf, "%4d", (int)(state->tally_idx)); + sprintf(buf, "%4d", display_val); watch_display_text(WATCH_POSITION_BOTTOM, buf); }