Refactor OSThread consumers to use disable()

src/concurrency/OSThread.cpp

@@ -93,10 +93,10 @@ void OSThread::run()
     currentThread = NULL;
 }
 
-void OSThread::disable() {
+void OSThread::disable() 
+{
     enabled = false;
     setInterval(INT32_MAX);
-
 }
 
 /**

src/input/RotaryEncoderInterruptImpl1.cpp

@@ -9,8 +9,7 @@ void RotaryEncoderInterruptImpl1::init()
 {
     if (!moduleConfig.canned_message.rotary1_enabled) {
         // Input device is disabled.
-        setInterval(INT32_MAX);
+        disable();
-        enabled = false;
         return;
     }
 

src/input/cardKbI2cImpl.cpp

@@ -12,9 +12,7 @@ void CardKbI2cImpl::init()
 {
     if (cardkb_found != CARDKB_ADDR)
     {
-        // Input device is not detected.
+        disable();
-        setInterval(INT32_MAX);
-        enabled = false;
         return;
     }
 

src/mesh/http/WebServer.cpp

@@ -167,16 +167,14 @@ WebServerThread *webServerThread;
 
 WebServerThread::WebServerThread() : concurrency::OSThread("WebServerThread") {
     if (!config.network.wifi_enabled) {
-        setInterval(INT32_MAX);
+        disable();
-        enabled = false;
     }
 }
 
 int32_t WebServerThread::runOnce()
 {
     if (!config.network.wifi_enabled) {
-        setInterval(INT32_MAX);
+        disable();
-        enabled = false;
     }
 
     handleWebResponse();

src/modules/CannedMessageModule.cpp

@@ -55,16 +55,14 @@ CannedMessageModule::CannedMessageModule()
         if ((this->splitConfiguredMessages() <= 0) && (cardkb_found != CARDKB_ADDR)) {
             DEBUG_MSG("CannedMessageModule: No messages are configured. Module is disabled\n");
             this->runState = CANNED_MESSAGE_RUN_STATE_DISABLED;
-            setInterval(INT32_MAX);
+            disable();
-            enabled = false;
         } else {
             DEBUG_MSG("CannedMessageModule is enabled\n");
             this->inputObserver.observe(inputBroker);
         }
     } else {
         this->runState = CANNED_MESSAGE_RUN_STATE_DISABLED;
-        setInterval(INT32_MAX);
+        disable();
-        enabled = false;
     }
 }
 

src/modules/ExternalNotificationModule.cpp

@@ -199,8 +199,7 @@ ExternalNotificationModule::ExternalNotificationModule()
         }
     } else {
         DEBUG_MSG("External Notification Module Disabled\n");
-        setInterval(INT32_MAX);
+        disable();
-        enabled = false;
     }
 }
 

src/modules/esp32/AudioModule.cpp

@@ -139,8 +139,7 @@ AudioModule::AudioModule() : SinglePortModule("AudioModule", PortNum_AUDIO_APP),
         DEBUG_MSG(" using %d frames of %d bytes for a total payload length of %d bytes\n", encode_frame_num, encode_codec_size, encode_frame_size);
         xTaskCreate(&run_codec2, "codec2_task", 30000, NULL, 5, &codec2HandlerTask);
     } else {
-        setInterval(INT32_MAX);
+        disable();
-        enabled = false;
         DEBUG_MSG("Codec2 disabled (AudioModule %d, Region %s, permitted %d)\n", moduleConfig.audio.codec2_enabled, myRegion->name, myRegion->audioPermitted);
     }
 }

src/modules/esp32/StoreForwardModule.cpp

@@ -460,8 +460,7 @@ StoreForwardModule::StoreForwardModule()
             DEBUG_MSG("*** Initializing Store & Forward Module in Client mode\n");
         }
     } else {
-        setInterval(INT32_MAX);
+        disable();
-        enabled = false;
     }
 #endif
 }

src/mqtt/MQTT.cpp

@@ -137,8 +137,7 @@ MQTT::MQTT() : concurrency::OSThread("mqtt"), pubSub(mqttClient), mqttQueue(MAX_
 
         // preflightSleepObserver.observe(&preflightSleep);
     } else {
-        setInterval(INT32_MAX);
+        disable();
-        enabled = false;
     }
 }