motioncontroller: Fix clang-tidy warnings

Also move one-line functions to header.

1 files changed, 11 insertions, 6 deletions

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;