From c566a9f46559b81a92e150aa94d752024fefe694 Mon Sep 17 00:00:00 2001 From: Blake Latchford Date: Sun, 17 Nov 2024 11:10:09 -0500 Subject: [PATCH] Centralize ExternalNotificationModule LED handling This PR attemtps to take a small step towards addressing issue 5194 by de-duplicating LED handling code within ExternalNotificationModule. I also refactored the color state code around an alpha channel approach. --- src/modules/ExternalNotificationModule.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/modules/ExternalNotificationModule.cpp b/src/modules/ExternalNotificationModule.cpp index 6f37d5535..d08badcc8 100644 --- a/src/modules/ExternalNotificationModule.cpp +++ b/src/modules/ExternalNotificationModule.cpp @@ -256,7 +256,8 @@ void ExternalNotificationModule::stopNow() void ExternalNotificationModule::setLEDs(const RGB& rgb) { -#ifdef HAS_LEDS + +#ifdef HAS_LED // LOG_DEBUG("setLEDs red=%d, green=%d, blue=%d", rgb.red, rgb.green, rgb.blue); #ifdef HAS_NCP5623