Logging can be removed from executable by not setting LOGGING symbol.

Author: JackKelly

Logger.cpp

@@ -1,5 +1,5 @@
 #include "Logger.h"
 
-#ifdef DEBUG
+#ifdef LOGGING
 Level Logger::log_threshold = WARN;
 #endif // DEBUG

Logger.h

@@ -7,17 +7,17 @@
 
 enum Level {DEBUG, INFO, WARN, ERROR, FATAL};
 
-#ifdef DEBUG
+#ifdef LOGGING
 class Logger {
 public:
     static Level log_threshold;
 };
-#endif // DEBUG
+#endif // LOGGING
 
 inline
 void print_log_level(const Level& level)
 {
-#ifdef DEBUG
+#ifdef LOGGING
     Serial.print(level);
 
     switch (level) {
@@ -33,20 +33,20 @@ void print_log_level(const Level& level)
 inline
 void print_log_levels()
 {
-#ifdef DEBUG
+#ifdef LOGGING
     for (uint8_t level=DEBUG; level<FATAL; level+=1) {
         print_log_level((Level)level);
         Serial.print(", ");
     }
     print_log_level(FATAL);
     Serial.println("");
-#endif //DEBUG
+#endif // DEBUG
 }
 
 
 inline void log(const Level& level, const char *__fmt, ...)
 {
-#ifdef DEBUG
+#ifdef LOGGING
     if (Logger::log_threshold <= level) {
         Serial.print(millis());
         Serial.print(" ");
@@ -62,7 +62,7 @@ inline void log(const Level& level, const char *__fmt, ...)
 
         Serial.println(buf);
     }
-#endif // DEBUG
+#endif // LOGGING
 }
 
 

Manager.cpp

@@ -69,16 +69,16 @@ void Manager::handle_serial_commands()
         }
         break;
     case 'v':
-#ifdef DEBUG
+#ifdef LOGGING
         Serial.println("ACK enter log level:");
         print_log_levels();
         Logger::log_threshold = (Level)utils::read_uint32_from_serial();
         Serial.print("ACK Log level set to ");
         print_log_level(Logger::log_threshold);
         Serial.println("");
 #else
-        Serial.println("NAK debugging disabled!");
-#endif // DEBUG
+        Serial.println("NAK logging disabled!");
+#endif // LOGGING
         break;
     case 'k': print_packets = ONLY_KNOWN; Serial.println("ACK only print data from known transmitters"); break;
     case 'u': print_packets = ALL_VALID; Serial.println("ACK print all valid packets"); break;