From c5650a55562272aa6f4e0bdb1b78a814128b306d Mon Sep 17 00:00:00 2001
From: Michele Bini <michele.bini@gmail.com>
Date: Sun, 10 Apr 2022 10:07:59 +0200
Subject: reduce conditional branching and tests for motion activity
 interactions


diff --git a/src/systemtask/SystemTask.cpp b/src/systemtask/SystemTask.cpp
index 0411cf2..20ff937 100644
--- a/src/systemtask/SystemTask.cpp
+++ b/src/systemtask/SystemTask.cpp
@@ -492,16 +492,18 @@ void SystemTask::UpdateMotion() {
 
   motionController.IsSensorOk(motionSensor.IsOk());
   motionController.Update(motionValues.x, motionValues.y, motionValues.z, motionValues.steps);
-  if (isSleeping && motionController.ShouldRaiseWake()) {
-    GoToRunning();
-  }
-  if (settingsController.isWakeUpModeOn(Pinetime::Controllers::Settings::WakeUpMode::Shake) &&
-      motionController.Should_ShakeWake(settingsController.GetShakeThreshold())) {
-    GoToRunning();
-  }
-  if (settingsController.isWakeUpModeOn(Pinetime::Controllers::Settings::WakeUpMode::LowerWrist) && !isSleeping &&
-      motionController.ShouldLowerSleep()) {
-    PushMessage(Messages::GoToSleep);
+  if (isSleeping) {
+    if (motionController.ShouldRaiseWake() ||
+        (settingsController.isWakeUpModeOn(Pinetime::Controllers::Settings::WakeUpMode::Shake) &&
+         motionController.Should_ShakeWake(settingsController.GetShakeThreshold()))
+       ) {
+      GoToRunning();
+    }
+  } else {
+    if (settingsController.isWakeUpModeOn(Pinetime::Controllers::Settings::WakeUpMode::LowerWrist) &&
+        motionController.ShouldLowerSleep()) {
+      PushMessage(Messages::GoToSleep);
+    }
   }
 }
 
-- 
cgit v0.10.2