Review every print to do it in 1 line instead of 2

Geoffroy Arenou · Copilot · Geoffroy Arenou · commit a27a91ea7f59 · 2026-05-06T14:50:04.000+02:00
Co-authored-by: Copilot &lt;copilot@github.com&gt;
diff --git a/include/ESP32_Helper.h b/include/ESP32_Helper.h
@@ -38,12 +38,6 @@
 
 namespace ESP32_Helper
 {
-    // Type definition for command handler function
-    // Returns true if the command was handled, false otherwise
-    typedef bool (*CommandHandlerFunc)(Command);
-    // Type definition for command help print function
-    typedef void (*CommandHelpFunc)();
-
     /**
      * Initialize serial/wifi for PC communication
      */
diff --git a/include/Structure_Helper.h b/include/Structure_Helper.h
@@ -396,6 +396,20 @@ struct Command
     }
 };
 
+// Type definition for command handler function
+// Returns true if the command was handled, false otherwise
+typedef bool (*CommandHandlerFunc)(Command);
+// Type definition for command help print function
+typedef void (*CommandHelpFunc)();
+
+// Used to store all custom command
+struct CommandHandler
+{
+    String prefix;
+    CommandHandlerFunc func;
+    CommandHelpFunc helpFunc;
+};
+
 // Use Start to specify the timeOut in ms,
 // Use IsTimeOut to check if the timer has passed the timeOut value
 struct Timeout
diff --git a/src/Debugger.cpp b/src/Debugger.cpp
@@ -32,13 +32,13 @@ namespace Debugger
 
     void Initialisation()
     {
-        Printer::print("Preparing queueSteps : ");
         queueSteps = QueueThread<int16_t>(100);
         if (!queueSteps.IsInit())
         {
-            Printer::println("Error creating the queueSteps !");
+            Printer::println("Preparing queueSteps : Error creating the queueSteps !");
+            return;
         }
-        Printer::println("done.");
+        Printer::println("Preparing queueSteps : done.");
     }
 
     bool HandleCommand(Command cmdTmp)
diff --git a/src/ESP32_Helper.cpp b/src/ESP32_Helper.cpp
@@ -7,13 +7,6 @@ namespace ESP32_Helper
 {
     namespace // anonymous nested namespace, cannot access outside this file
     {
-        struct CommandHandler
-        {
-            String prefix;
-            CommandHandlerFunc func;
-            CommandHelpFunc helpFunc;
-        };
-
         const int8_t readBufferMax = 64;
         std::vector<char> readBuffer;
         std::vector<CommandHandler> customHandlers;
@@ -74,9 +67,7 @@ namespace ESP32_Helper
         println("¦  M E C A P I  ¦");
         println("+---------------+");
         println();
-        print(__DATE__);
-        print(" at ");
-        println(__TIME__);
+        println("%s at %s", __DATE__, __TIME__);
         println();
 
         println("-- Starting Debugger Initialisation --");
@@ -129,8 +120,8 @@ namespace ESP32_Helper
             {
                 if (readBuffer.size() > 1)
                 {
-                    Printer::print("Received %u : ", static_cast<unsigned int>(readBuffer.size()));
-                    Printer::println(String(readBuffer.data(), readBuffer.size() - 1));
+                    String receivedCmd(readBuffer.data(), readBuffer.size() - 1);
+                    Printer::println("Received %u : %s", static_cast<unsigned int>(readBuffer.size()), receivedCmd.c_str());
                     ESP32_Helper::BufferReadCommand(readBuffer);
                 }
                 readBuffer.clear();
diff --git a/src/FileSystem_Helper.cpp b/src/FileSystem_Helper.cpp
@@ -31,8 +31,10 @@ namespace FileSystem_Helper
 
     void ListFiles(const String &filter)
     {
-        print("Listing files ");
-        filter != "" ? println(" with filter : <%s>", filter.c_str()) : println(" :");
+        if (filter != "")
+            println("Listing files with filter : <%s>", filter.c_str());
+        else
+            println("Listing files :");
 
         File root = SPIFFS.open("/");
         if (!root)
@@ -47,8 +49,7 @@ namespace FileSystem_Helper
         {
             if (filter == "" || (filter != "" && String(file.name()).indexOf(filter) != -1))
             {
-                print("  " + String(file.name()));
-                print(" \t " + String((int)file.size()) + "\n");
+                println("  %s \t %i", file.name(), (int)file.size());
             }
             file = root.openNextFile();
         }
@@ -166,8 +167,7 @@ namespace FileSystem_Helper
 
     void RenameFile(const String &path1, const String &path2)
     {
-        print("Renaming file ", path1);
-        println(" to file ", path2);
+        println("Renaming file %s to file %s", path1.c_str(), path2.c_str());
 
         if(!SPIFFS.exists("/" + path1))
         {
@@ -191,7 +191,7 @@ namespace FileSystem_Helper
 
     void DeleteFile(const String &fileName)
     {
-        println("Deleting file: ", fileName);
+        println("Deleting file: %s", fileName.c_str());
         
         if(!SPIFFS.exists("/" + fileName))
         {
diff --git a/src/Printer.cpp b/src/Printer.cpp
@@ -8,47 +8,60 @@ namespace Printer
     Enable teleplotUDPEnable = Enable::ENABLE_FALSE;
     namespace
     {
-        Level printLevel = Level::LEVEL_VERBOSE;
-        Enable printEnable = Enable::ENABLE_TRUE;
+        Level printLevel = Level::LEVEL_NONE;
+        Enable printEnable = Enable::ENABLE_NONE;
     }
 
     void SetLevel(Level level)
     {
         printLevel = level;
-        print("Printer Level : ");
+        const char *levelName = "LEVEL_UNKNOWN";
         switch (level)
         {
-            ENUM_PRINT(Level::LEVEL_VERBOSE);
-            ENUM_PRINT(Level::LEVEL_INFO);
-            ENUM_PRINT(Level::LEVEL_WARN);
-            ENUM_PRINT(Level::LEVEL_ERROR);
-            ENUM_PRINT(Level::LEVEL_NONE);
+            case Level::LEVEL_VERBOSE:
+                levelName = "LEVEL_VERBOSE";
+                break;
+            case Level::LEVEL_INFO:
+                levelName = "LEVEL_INFO";
+                break;
+            case Level::LEVEL_WARN:
+                levelName = "LEVEL_WARN";
+                break;
+            case Level::LEVEL_ERROR:
+                levelName = "LEVEL_ERROR";
+                break;
+            case Level::LEVEL_NONE:
+                levelName = "LEVEL_NONE";
+                break;
+            default:
+                break;
         }
+        println("Printer Level : %s", levelName);
     }
 
     void EnablePrinter(Enable enable)
     {
         if (enable != Enable::ENABLE_NONE)
         {
-            print("Printer : ");
+            const char *status = "unknown";
 
             if (printEnable == enable && enable == Enable::ENABLE_TRUE)
             {
-                print("already Enable");
+                status = "already Enable";
             }
             if (printEnable == enable && enable == Enable::ENABLE_FALSE)
             {
-                print("already Disable");
+                status = "already Disable";
             }
             if (printEnable != enable && enable == Enable::ENABLE_TRUE)
             {
-                print(" Enable");
+                status = "Enable";
             }
             if (printEnable != enable && enable == Enable::ENABLE_FALSE)
             {
-                print(" Disable");
+                status = "Disable";
             }
-            println();
+            println("Printer : %s", status);
             printEnable = enable;
         }
     }
@@ -136,28 +149,50 @@ namespace Printer
     void PrintLevel(Level level)
     {
         printLevel = level;
-        print("PrintLevel : ");
+        const char *levelName = "LEVEL_UNKNOWN";
         switch (level)
         {
-            ENUM_PRINT(Level::LEVEL_VERBOSE);
-            ENUM_PRINT(Level::LEVEL_INFO);
-            ENUM_PRINT(Level::LEVEL_WARN);
-            ENUM_PRINT(Level::LEVEL_ERROR);
-            ENUM_PRINT(Level::LEVEL_NONE);
+            case Level::LEVEL_VERBOSE:
+                levelName = "LEVEL_VERBOSE";
+                break;
+            case Level::LEVEL_INFO:
+                levelName = "LEVEL_INFO";
+                break;
+            case Level::LEVEL_WARN:
+                levelName = "LEVEL_WARN";
+                break;
+            case Level::LEVEL_ERROR:
+                levelName = "LEVEL_ERROR";
+                break;
+            case Level::LEVEL_NONE:
+                levelName = "LEVEL_NONE";
+                break;
+            default:
+                break;
         }
+        println("PrintLevel : %s", levelName);
     }
 
     Enable PrintEnable() { return printEnable; }
     void PrintEnable(Enable enable)
     {
         printEnable = enable;
-        print("PrintEnable : ");
+        const char *enableName = "ENABLE_UNKNOWN";
         switch (enable)
         {
-            ENUM_PRINT(Enable::ENABLE_FALSE);
-            ENUM_PRINT(Enable::ENABLE_TRUE);
-            ENUM_PRINT(Enable::ENABLE_NONE);
+            case Enable::ENABLE_FALSE:
+                enableName = "ENABLE_FALSE";
+                break;
+            case Enable::ENABLE_TRUE:
+                enableName = "ENABLE_TRUE";
+                break;
+            case Enable::ENABLE_NONE:
+                enableName = "ENABLE_NONE";
+                break;
+            default:
+                break;
         }
+        println("PrintEnable : %s", enableName);
     }
 
     /**
@@ -242,12 +277,9 @@ Format Specifier
     {
         if(Chrono::print)
         {
-            String name = "Chrono["+chrono.name+"]";
             int time = (int)(chrono.elapsedTime/chrono.loopNbr);
-            print("Chrono [%s]: %d µs", chrono.name.c_str(), time);
-                
             uint stackSize = uxTaskGetStackHighWaterMark(nullptr);
-            println("  --  Stack Size: %u", stackSize);
+            println("Chrono [%s]: %d µs  --  Stack Size: %u", chrono.name.c_str(), time, stackSize);
         }
         //if(Chrono::teleplot)
         //    teleplot(name,time);
diff --git a/src/Wifi_Helper.cpp b/src/Wifi_Helper.cpp
@@ -46,29 +46,29 @@ namespace Wifi_Helper
     {
         if (enable != Enable::ENABLE_NONE)
         {
-            Printer::print("Wifi : ");
+            const char *status = "unknown";
 
             if (wifiEnable == enable && enable == Enable::ENABLE_TRUE)
             {
-                Printer::print("already Enable");
+                status = "already Enable";
             }
             if (wifiEnable == enable && enable == Enable::ENABLE_FALSE)
             {
-                Printer::print("already Disable");
+                status = "already Disable";
                 wifiClient.stop();
                 WiFi.disconnect();
             }
             if (wifiEnable != enable && enable == Enable::ENABLE_TRUE)
             {
-                Printer::print(" Enable");
+                status = "Enable";
             }
             if (wifiEnable != enable && enable == Enable::ENABLE_FALSE)
             {
-                Printer::print(" Disable");
+                status = "Disable";
                 wifiClient.stop();
                 WiFi.disconnect();
             }
-            Printer::println();
+            Printer::println("Wifi : %s", status);
             wifiEnable = enable;
         }
     }
@@ -89,10 +89,8 @@ namespace Wifi_Helper
 
         WiFi.mode(WIFI_STA);
 
-        print("AP MAC : ");
-        println(WiFi.softAPmacAddress());
-        print("Wifi MAC : ");
-        println(WiFi.macAddress());
+        println("AP MAC : %s", WiFi.softAPmacAddress().c_str());
+        println("Wifi MAC : %s", WiFi.macAddress().c_str());
 
         // Get the Local Static IP saved
         String saved_wifi_local_ip = Preferences_Helper::LoadFromPreference("wifi_local_ip", "");

Original file line number	Diff line number	Diff line change
`@@ -32,13 +32,13 @@ namespace Debugger`
`32`	`32`
`33`	`33`	`void Initialisation()`
`34`	`34`	`{`
`35`		`- Printer::print("Preparing queueSteps : ");`
`36`	`35`	`queueSteps = QueueThread<int16_t>(100);`
`37`	`36`	`if (!queueSteps.IsInit())`
`38`	`37`	`{`
`39`		`- Printer::println("Error creating the queueSteps !");`
	`38`	`+ Printer::println("Preparing queueSteps : Error creating the queueSteps !");`
	`39`	`+ return;`
`40`	`40`	`}`
`41`		`- Printer::println("done.");`
	`41`	`+ Printer::println("Preparing queueSteps : done.");`
`42`	`42`	`}`
`43`	`43`
`44`	`44`	`bool HandleCommand(Command cmdTmp)`
Original file line number	Diff line number	Diff line change
`@@ -31,8 +31,10 @@ namespace FileSystem_Helper`
`31`	`31`
`32`	`32`	`void ListFiles(const String &filter)`
`33`	`33`	`{`
`34`		`- print("Listing files ");`
`35`		`- filter != "" ? println(" with filter : <%s>", filter.c_str()) : println(" :");`
	`34`	`+ if (filter != "")`
	`35`	`+ println("Listing files with filter : <%s>", filter.c_str());`
	`36`	`+ else`
	`37`	`+ println("Listing files :");`
`36`	`38`
`37`	`39`	`File root = SPIFFS.open("/");`
`38`	`40`	`if (!root)`
`@@ -47,8 +49,7 @@ namespace FileSystem_Helper`
`47`	`49`	`{`
`48`	`50`	`if (filter == "" \|\| (filter != "" && String(file.name()).indexOf(filter) != -1))`
`49`	`51`	`{`
`50`		`- print(" " + String(file.name()));`
`51`		`- print(" \t " + String((int)file.size()) + "\n");`
	`52`	`+ println(" %s \t %i", file.name(), (int)file.size());`
`52`	`53`	`}`
`53`	`54`	`file = root.openNextFile();`
`54`	`55`	`}`
`@@ -166,8 +167,7 @@ namespace FileSystem_Helper`
`166`	`167`
`167`	`168`	`void RenameFile(const String &path1, const String &path2)`
`168`	`169`	`{`
`169`		`- print("Renaming file ", path1);`
`170`		`- println(" to file ", path2);`
	`170`	`+ println("Renaming file %s to file %s", path1.c_str(), path2.c_str());`
`171`	`171`
`172`	`172`	`if(!SPIFFS.exists("/" + path1))`
`173`	`173`	`{`
`@@ -191,7 +191,7 @@ namespace FileSystem_Helper`
`191`	`191`
`192`	`192`	`void DeleteFile(const String &fileName)`
`193`	`193`	`{`
`194`		`- println("Deleting file: ", fileName);`
	`194`	`+ println("Deleting file: %s", fileName.c_str());`
`195`	`195`
`196`	`196`	`if(!SPIFFS.exists("/" + fileName))`
`197`	`197`	`{`
Original file line number	Diff line number	Diff line change
`@@ -8,47 +8,60 @@ namespace Printer`
`8`	`8`	`Enable teleplotUDPEnable = Enable::ENABLE_FALSE;`
`9`	`9`	`namespace`
`10`	`10`	`{`
`11`		`- Level printLevel = Level::LEVEL_VERBOSE;`
`12`		`- Enable printEnable = Enable::ENABLE_TRUE;`
	`11`	`+ Level printLevel = Level::LEVEL_NONE;`
	`12`	`+ Enable printEnable = Enable::ENABLE_NONE;`
`13`	`13`	`}`
`14`	`14`
`15`	`15`	`void SetLevel(Level level)`
`16`	`16`	`{`
`17`	`17`	`printLevel = level;`
`18`		`- print("Printer Level : ");`
	`18`	`+ const char *levelName = "LEVEL_UNKNOWN";`
`19`	`19`	`switch (level)`
`20`	`20`	`{`
`21`		`- ENUM_PRINT(Level::LEVEL_VERBOSE);`
`22`		`- ENUM_PRINT(Level::LEVEL_INFO);`
`23`		`- ENUM_PRINT(Level::LEVEL_WARN);`
`24`		`- ENUM_PRINT(Level::LEVEL_ERROR);`
`25`		`- ENUM_PRINT(Level::LEVEL_NONE);`
	`21`	`+ case Level::LEVEL_VERBOSE:`
	`22`	`+ levelName = "LEVEL_VERBOSE";`
	`23`	`+ break;`
	`24`	`+ case Level::LEVEL_INFO:`
	`25`	`+ levelName = "LEVEL_INFO";`
	`26`	`+ break;`
	`27`	`+ case Level::LEVEL_WARN:`
	`28`	`+ levelName = "LEVEL_WARN";`
	`29`	`+ break;`
	`30`	`+ case Level::LEVEL_ERROR:`
	`31`	`+ levelName = "LEVEL_ERROR";`
	`32`	`+ break;`
	`33`	`+ case Level::LEVEL_NONE:`
	`34`	`+ levelName = "LEVEL_NONE";`
	`35`	`+ break;`
	`36`	`+ default:`
	`37`	`+ break;`
`26`	`38`	`}`
	`39`	`+ println("Printer Level : %s", levelName);`
`27`	`40`	`}`
`28`	`41`
`29`	`42`	`void EnablePrinter(Enable enable)`
`30`	`43`	`{`
`31`	`44`	`if (enable != Enable::ENABLE_NONE)`
`32`	`45`	`{`
`33`		`- print("Printer : ");`
	`46`	`+ const char *status = "unknown";`
`34`	`47`
`35`	`48`	`if (printEnable == enable && enable == Enable::ENABLE_TRUE)`
`36`	`49`	`{`
`37`		`- print("already Enable");`
	`50`	`+ status = "already Enable";`
`38`	`51`	`}`
`39`	`52`	`if (printEnable == enable && enable == Enable::ENABLE_FALSE)`
`40`	`53`	`{`
`41`		`- print("already Disable");`
	`54`	`+ status = "already Disable";`
`42`	`55`	`}`
`43`	`56`	`if (printEnable != enable && enable == Enable::ENABLE_TRUE)`
`44`	`57`	`{`
`45`		`- print(" Enable");`
	`58`	`+ status = "Enable";`
`46`	`59`	`}`
`47`	`60`	`if (printEnable != enable && enable == Enable::ENABLE_FALSE)`
`48`	`61`	`{`
`49`		`- print(" Disable");`
	`62`	`+ status = "Disable";`
`50`	`63`	`}`
`51`		`- println();`
	`64`	`+ println("Printer : %s", status);`
`52`	`65`	`printEnable = enable;`
`53`	`66`	`}`
`54`	`67`	`}`
`@@ -136,28 +149,50 @@ namespace Printer`
`136`	`149`	`void PrintLevel(Level level)`
`137`	`150`	`{`
`138`	`151`	`printLevel = level;`
`139`		`- print("PrintLevel : ");`
	`152`	`+ const char *levelName = "LEVEL_UNKNOWN";`
`140`	`153`	`switch (level)`
`141`	`154`	`{`
`142`		`- ENUM_PRINT(Level::LEVEL_VERBOSE);`
`143`		`- ENUM_PRINT(Level::LEVEL_INFO);`
`144`		`- ENUM_PRINT(Level::LEVEL_WARN);`
`145`		`- ENUM_PRINT(Level::LEVEL_ERROR);`
`146`		`- ENUM_PRINT(Level::LEVEL_NONE);`
	`155`	`+ case Level::LEVEL_VERBOSE:`
	`156`	`+ levelName = "LEVEL_VERBOSE";`
	`157`	`+ break;`
	`158`	`+ case Level::LEVEL_INFO:`
	`159`	`+ levelName = "LEVEL_INFO";`
	`160`	`+ break;`
	`161`	`+ case Level::LEVEL_WARN:`
	`162`	`+ levelName = "LEVEL_WARN";`
	`163`	`+ break;`
	`164`	`+ case Level::LEVEL_ERROR:`
	`165`	`+ levelName = "LEVEL_ERROR";`
	`166`	`+ break;`
	`167`	`+ case Level::LEVEL_NONE:`
	`168`	`+ levelName = "LEVEL_NONE";`
	`169`	`+ break;`
	`170`	`+ default:`
	`171`	`+ break;`
`147`	`172`	`}`
	`173`	`+ println("PrintLevel : %s", levelName);`
`148`	`174`	`}`
`149`	`175`
`150`	`176`	`Enable PrintEnable() { return printEnable; }`
`151`	`177`	`void PrintEnable(Enable enable)`
`152`	`178`	`{`
`153`	`179`	`printEnable = enable;`
`154`		`- print("PrintEnable : ");`
	`180`	`+ const char *enableName = "ENABLE_UNKNOWN";`
`155`	`181`	`switch (enable)`
`156`	`182`	`{`
`157`		`- ENUM_PRINT(Enable::ENABLE_FALSE);`
`158`		`- ENUM_PRINT(Enable::ENABLE_TRUE);`
`159`		`- ENUM_PRINT(Enable::ENABLE_NONE);`
	`183`	`+ case Enable::ENABLE_FALSE:`
	`184`	`+ enableName = "ENABLE_FALSE";`
	`185`	`+ break;`
	`186`	`+ case Enable::ENABLE_TRUE:`
	`187`	`+ enableName = "ENABLE_TRUE";`
	`188`	`+ break;`
	`189`	`+ case Enable::ENABLE_NONE:`
	`190`	`+ enableName = "ENABLE_NONE";`
	`191`	`+ break;`
	`192`	`+ default:`
	`193`	`+ break;`
`160`	`194`	`}`
	`195`	`+ println("PrintEnable : %s", enableName);`
`161`	`196`	`}`
`162`	`197`
`163`	`198`	`/**`
`@@ -242,12 +277,9 @@ Format Specifier`
`242`	`277`	`{`
`243`	`278`	`if(Chrono::print)`
`244`	`279`	`{`
`245`		`- String name = "Chrono["+chrono.name+"]";`
`246`	`280`	`int time = (int)(chrono.elapsedTime/chrono.loopNbr);`
`247`		`- print("Chrono [%s]: %d µs", chrono.name.c_str(), time);`
`248`		`-`
`249`	`281`	`uint stackSize = uxTaskGetStackHighWaterMark(nullptr);`
`250`		`- println(" -- Stack Size: %u", stackSize);`
	`282`	`+ println("Chrono [%s]: %d µs -- Stack Size: %u", chrono.name.c_str(), time, stackSize);`
`251`	`283`	`}`
`252`	`284`	`//if(Chrono::teleplot)`
`253`	`285`	`// teleplot(name,time);`
Original file line number	Diff line number	Diff line change
`@@ -46,29 +46,29 @@ namespace Wifi_Helper`
`46`	`46`	`{`
`47`	`47`	`if (enable != Enable::ENABLE_NONE)`
`48`	`48`	`{`
`49`		`- Printer::print("Wifi : ");`
	`49`	`+ const char *status = "unknown";`
`50`	`50`
`51`	`51`	`if (wifiEnable == enable && enable == Enable::ENABLE_TRUE)`
`52`	`52`	`{`
`53`		`- Printer::print("already Enable");`
	`53`	`+ status = "already Enable";`
`54`	`54`	`}`
`55`	`55`	`if (wifiEnable == enable && enable == Enable::ENABLE_FALSE)`
`56`	`56`	`{`
`57`		`- Printer::print("already Disable");`
	`57`	`+ status = "already Disable";`
`58`	`58`	`wifiClient.stop();`
`59`	`59`	`WiFi.disconnect();`
`60`	`60`	`}`
`61`	`61`	`if (wifiEnable != enable && enable == Enable::ENABLE_TRUE)`
`62`	`62`	`{`
`63`		`- Printer::print(" Enable");`
	`63`	`+ status = "Enable";`
`64`	`64`	`}`
`65`	`65`	`if (wifiEnable != enable && enable == Enable::ENABLE_FALSE)`
`66`	`66`	`{`
`67`		`- Printer::print(" Disable");`
	`67`	`+ status = "Disable";`
`68`	`68`	`wifiClient.stop();`
`69`	`69`	`WiFi.disconnect();`
`70`	`70`	`}`
`71`		`- Printer::println();`
	`71`	`+ Printer::println("Wifi : %s", status);`
`72`	`72`	`wifiEnable = enable;`
`73`	`73`	`}`
`74`	`74`	`}`
`@@ -89,10 +89,8 @@ namespace Wifi_Helper`
`89`	`89`
`90`	`90`	`WiFi.mode(WIFI_STA);`
`91`	`91`
`92`		`- print("AP MAC : ");`
`93`		`- println(WiFi.softAPmacAddress());`
`94`		`- print("Wifi MAC : ");`
`95`		`- println(WiFi.macAddress());`
	`92`	`+ println("AP MAC : %s", WiFi.softAPmacAddress().c_str());`
	`93`	`+ println("Wifi MAC : %s", WiFi.macAddress().c_str());`
`96`	`94`
`97`	`95`	`// Get the Local Static IP saved`
`98`	`96`	`String saved_wifi_local_ip = Preferences_Helper::LoadFromPreference("wifi_local_ip", "");`