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.cpp | 10 +---------
 1 file changed, 1 insertion(+), 9 deletions(-)

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

diff --git a/src/components/motion/MotionController.cpp b/src/components/motion/MotionController.cpp
index 5fc548be..5087504c 100644
--- a/src/components/motion/MotionController.cpp
+++ b/src/components/motion/MotionController.cpp
@@ -23,10 +23,10 @@ void MotionController::Update(int16_t x, int16_t y, int16_t z, uint32_t nbSteps)
   this->z = z;
 
   int32_t deltaSteps = nbSteps - this->nbSteps;
-  this->nbSteps = nbSteps;
   if (deltaSteps > 0) {
     currentTripSteps += deltaSteps;
   }
+  this->nbSteps = nbSteps;
 }
 
 bool MotionController::Should_RaiseWake(bool isSleeping) {
@@ -61,10 +61,6 @@ bool MotionController::ShouldShakeWake(uint16_t thresh) {
   return accumulatedSpeed > thresh;
 }
 
-void MotionController::IsSensorOk(bool isOk) {
-  isSensorOk = isOk;
-}
-
 void MotionController::Init(Pinetime::Drivers::Bma421::DeviceTypes types) {
   switch (types) {
     case Drivers::Bma421::DeviceTypes::BMA421:
@@ -78,7 +74,3 @@ void MotionController::Init(Pinetime::Drivers::Bma421::DeviceTypes types) {
       break;
   }
 }
-
-void MotionController::SetService(Pinetime::Controllers::MotionService* service) {
-  this->service = service;
-}
-- 
cgit v1.2.3-70-g09d2