From 49ad5be7423f983804e65dd1571cc5dcea26babc Mon Sep 17 00:00:00 2001
From: Finlay Davidson <finlay.davidson@coderclass.nl>
Date: Sun, 5 Mar 2023 15:24:06 +0100
Subject: motioncontroller: Fix clang-tidy warnings

Also move one-line functions to header.
---
 src/components/motion/MotionController.h | 17 +++++++++++------
 1 file changed, 11 insertions(+), 6 deletions(-)

(limited to 'src/components/motion/MotionController.h')

diff --git a/src/components/motion/MotionController.h b/src/components/motion/MotionController.h
index 126504a3..fce8b53f 100644
--- a/src/components/motion/MotionController.h
+++ b/src/components/motion/MotionController.h
@@ -50,7 +50,9 @@ namespace Pinetime {
         return accumulatedSpeed;
       }
 
-      void IsSensorOk(bool isOk);
+      void IsSensorOk(bool isOk) {
+        isSensorOk = isOk;
+      }
 
       bool IsSensorOk() const {
         return isSensorOk;
@@ -61,21 +63,24 @@ namespace Pinetime {
       }
 
       void Init(Pinetime::Drivers::Bma421::DeviceTypes types);
-      void SetService(Pinetime::Controllers::MotionService* service);
+
+      void SetService(Pinetime::Controllers::MotionService* service) {
+        this->service = service;
+      }
 
     private:
-      uint32_t nbSteps;
+      uint32_t nbSteps = 0;
       uint32_t currentTripSteps = 0;
 
       TickType_t lastTime = 0;
       TickType_t time = 0;
 
-      int16_t x;
+      int16_t x = 0;
       int16_t lastYForWakeUp = 0;
       int16_t lastY = 0;
-      int16_t y;
+      int16_t y = 0;
       int16_t lastZ = 0;
-      int16_t z;
+      int16_t z = 0;
       int32_t accumulatedSpeed = 0;
 
       bool isSensorOk = false;
-- 
cgit v1.2.3-70-g09d2