From c7c5671cca8e996a1b04465f71e39b51f40a3ca6 Mon Sep 17 00:00:00 2001
From: Ben Meadors <benmmeadors@gmail.com>
Date: Thu, 29 Dec 2022 18:48:33 -0600
Subject: [PATCH] More disables

---
 src/modules/RemoteHardwareModule.cpp           | 3 +--
 src/modules/SerialModule.cpp                   | 4 +---
 src/modules/Telemetry/EnvironmentTelemetry.cpp | 3 +--
 src/modules/esp32/RangeTestModule.cpp          | 3 +--
 src/modules/esp32/StoreForwardModule.cpp       | 3 +--
 5 files changed, 5 insertions(+), 11 deletions(-)

diff --git a/src/modules/RemoteHardwareModule.cpp b/src/modules/RemoteHardwareModule.cpp
index 419885822..f8f2fecae 100644
--- a/src/modules/RemoteHardwareModule.cpp
+++ b/src/modules/RemoteHardwareModule.cpp
@@ -135,8 +135,7 @@ int32_t RemoteHardwareModule::runOnce()
         }
     } else {
         // No longer watching anything - stop using CPU
-        enabled = false;
-        return INT32_MAX;
+        return disable();
     }
 
     return 200; // Poll our GPIOs every 200ms (FIXME, make adjustable via protobuf arg)
diff --git a/src/modules/SerialModule.cpp b/src/modules/SerialModule.cpp
index 00d8df9d2..fc6448a97 100644
--- a/src/modules/SerialModule.cpp
+++ b/src/modules/SerialModule.cpp
@@ -221,9 +221,7 @@ int32_t SerialModule::runOnce()
         return (10);
     } else {
         DEBUG_MSG("Serial Module Disabled\n");
-
-        enabled = false;
-        return INT32_MAX;
+        return disable();
     }
 }
 
diff --git a/src/modules/Telemetry/EnvironmentTelemetry.cpp b/src/modules/Telemetry/EnvironmentTelemetry.cpp
index 500acd7f7..b2b100257 100644
--- a/src/modules/Telemetry/EnvironmentTelemetry.cpp
+++ b/src/modules/Telemetry/EnvironmentTelemetry.cpp
@@ -67,8 +67,7 @@ int32_t EnvironmentTelemetryModule::runOnce()
     if (!(moduleConfig.telemetry.environment_measurement_enabled ||
           moduleConfig.telemetry.environment_screen_enabled)) {
         // If this module is not enabled, and the user doesn't want the display screen don't waste any OSThread time on it
-        enabled = false;
-        return result;
+        return disable();
     }
 
     if (firstTime) {
diff --git a/src/modules/esp32/RangeTestModule.cpp b/src/modules/esp32/RangeTestModule.cpp
index 5e52fea40..0aa25946d 100644
--- a/src/modules/esp32/RangeTestModule.cpp
+++ b/src/modules/esp32/RangeTestModule.cpp
@@ -82,8 +82,7 @@ int32_t RangeTestModule::runOnce()
 
                 return (senderHeartbeat);
             } else {
-                enabled = false;
-                return (INT32_MAX);
+                return disable();
                 // This thread does not need to run as a receiver
             }
 
diff --git a/src/modules/esp32/StoreForwardModule.cpp b/src/modules/esp32/StoreForwardModule.cpp
index af0e60ed7..70a28ac68 100644
--- a/src/modules/esp32/StoreForwardModule.cpp
+++ b/src/modules/esp32/StoreForwardModule.cpp
@@ -52,8 +52,7 @@ int32_t StoreForwardModule::runOnce()
         return (this->packetTimeMax);
     }
 #endif
-    enabled = false; // Client doesn't need periodical
-    return (INT32_MAX);
+    return disable();
 }
 
 /*