From: Matheus Afonso Martins Moreira <matheus.a.m.moreira@gmail.com>
Date: Sat, 24 Feb 2024 08:17:56 +0000 (-0300)
Subject: faces/pulsometer: move structure definition
X-Git-Url: https://git.earman.xyz/?a=commitdiff_plain;h=30ebf4743e3693bc0740a479e9fa6460804fede4;p=sensor-watch.git

faces/pulsometer: move structure definition

Instances of the pulsometer state structure are only passed
to the pulsometer itself and only via the opaque context pointer.
No other code uses it. There is no need to expose it in a header file
so make it an implementation detail of the watch face.
---

diff --git a/movement/watch_faces/complication/pulsometer_face.c b/movement/watch_faces/complication/pulsometer_face.c
index 5e0209b..8c5f1cd 100644
--- a/movement/watch_faces/complication/pulsometer_face.c
+++ b/movement/watch_faces/complication/pulsometer_face.c
@@ -51,6 +51,13 @@
 
 #define PULSOMETER_FACE_FREQUENCY (1 << PULSOMETER_FACE_FREQUENCY_FACTOR)
 
+typedef struct {
+    bool measuring;
+    int16_t pulses;
+    int16_t ticks;
+    int8_t calibration;
+} pulsometer_state_t;
+
 static void pulsometer_display_title(pulsometer_state_t *pulsometer) {
     watch_display_string(PULSOMETER_FACE_TITLE, 0);
 }
diff --git a/movement/watch_faces/complication/pulsometer_face.h b/movement/watch_faces/complication/pulsometer_face.h
index ba629f2..5c1dae9 100644
--- a/movement/watch_faces/complication/pulsometer_face.h
+++ b/movement/watch_faces/complication/pulsometer_face.h
@@ -71,13 +71,6 @@
 
 #include "movement.h"
 
-typedef struct {
-    bool measuring;
-    int16_t pulses;
-    int16_t ticks;
-    int8_t calibration;
-} pulsometer_state_t;
-
 void pulsometer_face_setup(movement_settings_t *settings, uint8_t watch_face_index, void ** context_ptr);
 void pulsometer_face_activate(movement_settings_t *settings, void *context);
 bool pulsometer_face_loop(movement_event_t event, movement_settings_t *settings, void *context);