From d6ea68db9424a71febdd8bccd6f940f6f090f576 Mon Sep 17 00:00:00 2001
From: John <ebusd@ebusd.eu>
Date: Sat, 24 Feb 2024 13:25:07 +0100
Subject: [PATCH] simplify log facility handling

---
 src/ebusd/main.cpp    | 2 +-
 src/lib/utils/log.cpp | 8 ++------
 src/lib/utils/log.h   | 5 +----
 3 files changed, 4 insertions(+), 11 deletions(-)

diff --git a/src/ebusd/main.cpp b/src/ebusd/main.cpp
index 6e5237852..9d39b8a77 100644
--- a/src/ebusd/main.cpp
+++ b/src/ebusd/main.cpp
@@ -256,7 +256,7 @@ int main(int argc, char* argv[], char* envp[]) {
   }
 
   if (s_opt.logAreas != -1 || s_opt.logLevel != ll_COUNT) {
-    setFacilitiesLogLevel(LF_ALL, ll_none);
+    setFacilitiesLogLevel(1<<ll_COUNT, ll_none);
     setFacilitiesLogLevel(s_opt.logAreas, s_opt.logLevel);
   }
 
diff --git a/src/lib/utils/log.cpp b/src/lib/utils/log.cpp
index ed03f6def..5e04099c9 100755
--- a/src/lib/utils/log.cpp
+++ b/src/lib/utils/log.cpp
@@ -103,11 +103,7 @@ int parseLogFacilities(const char* facilities) {
       free(input);
       return -1;
     }
-    if (val == lf_COUNT) {
-      newFacilites = LF_ALL;
-    } else {
-      newFacilites |= 1 << val;
-    }
+    newFacilites |= 1 << val;
   }
   free(input);
   return newFacilites;
@@ -139,7 +135,7 @@ const char* getLogLevelStr(LogLevel level) {
 bool setFacilitiesLogLevel(int facilities, LogLevel level) {
   bool changed = false;
   for (int val = 0; val < lf_COUNT && facilities != 0; val++) {
-    if ((facilities & (1 << val)) != 0 && s_facilityLogLevel[(LogFacility)val] != level) {
+    if ((facilities & ((1 << val)|(1 << lf_COUNT))) != 0 && s_facilityLogLevel[(LogFacility)val] != level) {
       s_facilityLogLevel[(LogFacility)val] = level;
       changed = true;
     }
diff --git a/src/lib/utils/log.h b/src/lib/utils/log.h
index 1a4d78ba1..221fc1144 100755
--- a/src/lib/utils/log.h
+++ b/src/lib/utils/log.h
@@ -30,12 +30,9 @@ enum LogFacility {
   lf_bus,       //!< eBUS related
   lf_update,    //!< updates found while listening to the bus
   lf_other,     //!< all other log facilities
-  lf_COUNT = 5  //!< number of available log facilities
+  lf_COUNT = 5  //!< number of available log facilities and flag for setting all
 };
 
-/** macro for all log facilities. */
-#define LF_ALL ((1 << lf_main) | (1 << lf_network) | (1 << lf_bus) | (1 << lf_update) | (1 << lf_other))
-
 /** the available log levels. */
 enum LogLevel {
   ll_none = 0,  //!< no level at all