motioncontroller: Fix clang-tidy warnings

Also move one-line functions to header.

1 files changed, 1 insertions, 9 deletions

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;
-}