new newlines for libserialport log messages

mkalkbrenner · mkalkbrenner · commit de98ff2baa19 · 2025-02-11T15:25:44.000+01:00
diff --git a/src/ZeDMDComm.cpp b/src/ZeDMDComm.cpp
@@ -360,7 +360,7 @@ bool ZeDMDComm::Connect()
         const char* error_msg = sp_last_error_message();
         if (error_msg)
         {
-          Log("libserialport error: %s\n", error_msg);
+          Log("libserialport error: %s", error_msg);
         }
       }
 
@@ -407,7 +407,7 @@ bool ZeDMDComm::Connect(char* pDevice)
     const char* error_msg = sp_last_error_message();
     if (error_msg)
     {
-      Log("libserialport error: %s\n", error_msg);
+      Log("libserialport error: %s", error_msg);
     }
   }
 
@@ -429,7 +429,7 @@ bool ZeDMDComm::Connect(char* pDevice)
         const char* error_msg = sp_last_error_message();
         if (error_msg)
         {
-          Log("libserialport error: %s\n", error_msg);
+          Log("libserialport error: %s", error_msg);
         }
       }
 
@@ -485,7 +485,7 @@ bool ZeDMDComm::Connect(char* pDevice)
     const char* error_msg = sp_last_error_message();
     if (error_msg)
     {
-      Log("libserialport error: %s\n", error_msg);
+      Log("libserialport error: %s", error_msg);
     }
     Log("Unable to open device %s, error code %d", pDevice, result);
     sp_free_port(m_pSerialPort);
@@ -498,7 +498,7 @@ bool ZeDMDComm::Connect(char* pDevice)
     const char* error_msg = sp_last_error_message();
     if (error_msg)
     {
-      Log("libserialport error: %s\n", error_msg);
+      Log("libserialport error: %s", error_msg);
     }
     Log("Unable to set baudrate on device %s, error code %d", pDevice, result);
     sp_free_port(m_pSerialPort);
@@ -511,7 +511,7 @@ bool ZeDMDComm::Connect(char* pDevice)
     const char* error_msg = sp_last_error_message();
     if (error_msg)
     {
-      Log("libserialport error: %s\n", error_msg);
+      Log("libserialport error: %s", error_msg);
     }
     Log("Unable to set bits on device %s, error code %d", pDevice, result);
     sp_free_port(m_pSerialPort);
@@ -524,7 +524,7 @@ bool ZeDMDComm::Connect(char* pDevice)
     const char* error_msg = sp_last_error_message();
     if (error_msg)
     {
-      Log("libserialport error: %s\n", error_msg);
+      Log("libserialport error: %s", error_msg);
     }
     Log("Unable to set parity on device %s, error code %d", pDevice, result);
     sp_free_port(m_pSerialPort);
@@ -537,7 +537,7 @@ bool ZeDMDComm::Connect(char* pDevice)
     const char* error_msg = sp_last_error_message();
     if (error_msg)
     {
-      Log("libserialport error: %s\n", error_msg);
+      Log("libserialport error: %s", error_msg);
     }
     Log("Unable to to set stopbits on device %s, error code %d", pDevice, result);
     sp_free_port(m_pSerialPort);
@@ -550,7 +550,7 @@ bool ZeDMDComm::Connect(char* pDevice)
     const char* error_msg = sp_last_error_message();
     if (error_msg)
     {
-      Log("libserialport error: %s\n", error_msg);
+      Log("libserialport error: %s", error_msg);
     }
     Log("Unable to set xon xoff on device %s, error code %d", pDevice, result);
     sp_free_port(m_pSerialPort);
@@ -563,7 +563,7 @@ bool ZeDMDComm::Connect(char* pDevice)
     const char* error_msg = sp_last_error_message();
     if (error_msg)
     {
-      Log("libserialport error: %s\n", error_msg);
+      Log("libserialport error: %s", error_msg);
     }
     Log("Unable to set flowcontrol on device %s, error code %d", pDevice, result);
     sp_free_port(m_pSerialPort);
@@ -600,7 +600,7 @@ bool ZeDMDComm::Handshake(char* pDevice)
       const char* error_msg = sp_last_error_message();
       if (error_msg)
       {
-        Log("libserialport error: %s\n", error_msg);
+        Log("libserialport error: %s", error_msg);
       }
     }
 
@@ -611,7 +611,7 @@ bool ZeDMDComm::Handshake(char* pDevice)
         const char* error_msg = sp_last_error_message();
         if (error_msg)
         {
-          Log("libserialport error: %s\n", error_msg);
+          Log("libserialport error: %s", error_msg);
         }
       }
       std::this_thread::sleep_for(std::chrono::milliseconds(2));
@@ -692,7 +692,7 @@ bool ZeDMDComm::Handshake(char* pDevice)
           const char* error_msg = sp_last_error_message();
           if (error_msg)
           {
-            Log("libserialport error: %s\n", error_msg);
+            Log("libserialport error: %s", error_msg);
           }
         }
 
@@ -706,7 +706,7 @@ bool ZeDMDComm::Handshake(char* pDevice)
         const char* error_msg = sp_last_error_message();
         if (error_msg)
         {
-          Log("libserialport error: %s\n", error_msg);
+          Log("libserialport error: %s", error_msg);
         }
       }
 
@@ -876,7 +876,7 @@ bool ZeDMDComm::SendChunks(uint8_t* pData, uint16_t size)
       const char* error_msg = sp_last_error_message();
       if (error_msg)
       {
-        Log("libserialport error: %s\n", error_msg);
+        Log("libserialport error: %s", error_msg);
       }
     }
   }
@@ -919,7 +919,7 @@ bool ZeDMDComm::SendChunks(uint8_t* pData, uint16_t size)
         const char* error_msg = sp_last_error_message();
         if (error_msg)
         {
-          Log("libserialport error: %s\n", error_msg);
+          Log("libserialport error: %s", error_msg);
         }
       }
 
@@ -955,7 +955,7 @@ bool ZeDMDComm::SendChunks(uint8_t* pData, uint16_t size)
           const char* error_msg = sp_last_error_message();
           if (error_msg)
           {
-            Log("libserialport error: %s\n", error_msg);
+            Log("libserialport error: %s", error_msg);
           }
         }
 

Original file line number	Diff line number	Diff line change
`@@ -360,7 +360,7 @@ bool ZeDMDComm::Connect()`
`360`	`360`	`const char* error_msg = sp_last_error_message();`
`361`	`361`	`if (error_msg)`
`362`	`362`	`{`
`363`		`- Log("libserialport error: %s\n", error_msg);`
	`363`	`+ Log("libserialport error: %s", error_msg);`
`364`	`364`	`}`
`365`	`365`	`}`
`366`	`366`
`@@ -407,7 +407,7 @@ bool ZeDMDComm::Connect(char* pDevice)`
`407`	`407`	`const char* error_msg = sp_last_error_message();`
`408`	`408`	`if (error_msg)`
`409`	`409`	`{`
`410`		`- Log("libserialport error: %s\n", error_msg);`
	`410`	`+ Log("libserialport error: %s", error_msg);`
`411`	`411`	`}`
`412`	`412`	`}`
`413`	`413`
`@@ -429,7 +429,7 @@ bool ZeDMDComm::Connect(char* pDevice)`
`429`	`429`	`const char* error_msg = sp_last_error_message();`
`430`	`430`	`if (error_msg)`
`431`	`431`	`{`
`432`		`- Log("libserialport error: %s\n", error_msg);`
	`432`	`+ Log("libserialport error: %s", error_msg);`
`433`	`433`	`}`
`434`	`434`	`}`
`435`	`435`
`@@ -485,7 +485,7 @@ bool ZeDMDComm::Connect(char* pDevice)`
`485`	`485`	`const char* error_msg = sp_last_error_message();`
`486`	`486`	`if (error_msg)`
`487`	`487`	`{`
`488`		`- Log("libserialport error: %s\n", error_msg);`
	`488`	`+ Log("libserialport error: %s", error_msg);`
`489`	`489`	`}`
`490`	`490`	`Log("Unable to open device %s, error code %d", pDevice, result);`
`491`	`491`	`sp_free_port(m_pSerialPort);`
`@@ -498,7 +498,7 @@ bool ZeDMDComm::Connect(char* pDevice)`
`498`	`498`	`const char* error_msg = sp_last_error_message();`
`499`	`499`	`if (error_msg)`
`500`	`500`	`{`
`501`		`- Log("libserialport error: %s\n", error_msg);`
	`501`	`+ Log("libserialport error: %s", error_msg);`
`502`	`502`	`}`
`503`	`503`	`Log("Unable to set baudrate on device %s, error code %d", pDevice, result);`
`504`	`504`	`sp_free_port(m_pSerialPort);`
`@@ -511,7 +511,7 @@ bool ZeDMDComm::Connect(char* pDevice)`
`511`	`511`	`const char* error_msg = sp_last_error_message();`
`512`	`512`	`if (error_msg)`
`513`	`513`	`{`
`514`		`- Log("libserialport error: %s\n", error_msg);`
	`514`	`+ Log("libserialport error: %s", error_msg);`
`515`	`515`	`}`
`516`	`516`	`Log("Unable to set bits on device %s, error code %d", pDevice, result);`
`517`	`517`	`sp_free_port(m_pSerialPort);`
`@@ -524,7 +524,7 @@ bool ZeDMDComm::Connect(char* pDevice)`
`524`	`524`	`const char* error_msg = sp_last_error_message();`
`525`	`525`	`if (error_msg)`
`526`	`526`	`{`
`527`		`- Log("libserialport error: %s\n", error_msg);`
	`527`	`+ Log("libserialport error: %s", error_msg);`
`528`	`528`	`}`
`529`	`529`	`Log("Unable to set parity on device %s, error code %d", pDevice, result);`
`530`	`530`	`sp_free_port(m_pSerialPort);`
`@@ -537,7 +537,7 @@ bool ZeDMDComm::Connect(char* pDevice)`
`537`	`537`	`const char* error_msg = sp_last_error_message();`
`538`	`538`	`if (error_msg)`
`539`	`539`	`{`
`540`		`- Log("libserialport error: %s\n", error_msg);`
	`540`	`+ Log("libserialport error: %s", error_msg);`
`541`	`541`	`}`
`542`	`542`	`Log("Unable to to set stopbits on device %s, error code %d", pDevice, result);`
`543`	`543`	`sp_free_port(m_pSerialPort);`
`@@ -550,7 +550,7 @@ bool ZeDMDComm::Connect(char* pDevice)`
`550`	`550`	`const char* error_msg = sp_last_error_message();`
`551`	`551`	`if (error_msg)`
`552`	`552`	`{`
`553`		`- Log("libserialport error: %s\n", error_msg);`
	`553`	`+ Log("libserialport error: %s", error_msg);`
`554`	`554`	`}`
`555`	`555`	`Log("Unable to set xon xoff on device %s, error code %d", pDevice, result);`
`556`	`556`	`sp_free_port(m_pSerialPort);`
`@@ -563,7 +563,7 @@ bool ZeDMDComm::Connect(char* pDevice)`
`563`	`563`	`const char* error_msg = sp_last_error_message();`
`564`	`564`	`if (error_msg)`
`565`	`565`	`{`
`566`		`- Log("libserialport error: %s\n", error_msg);`
	`566`	`+ Log("libserialport error: %s", error_msg);`
`567`	`567`	`}`
`568`	`568`	`Log("Unable to set flowcontrol on device %s, error code %d", pDevice, result);`
`569`	`569`	`sp_free_port(m_pSerialPort);`
`@@ -600,7 +600,7 @@ bool ZeDMDComm::Handshake(char* pDevice)`
`600`	`600`	`const char* error_msg = sp_last_error_message();`
`601`	`601`	`if (error_msg)`
`602`	`602`	`{`
`603`		`- Log("libserialport error: %s\n", error_msg);`
	`603`	`+ Log("libserialport error: %s", error_msg);`
`604`	`604`	`}`
`605`	`605`	`}`
`606`	`606`
`@@ -611,7 +611,7 @@ bool ZeDMDComm::Handshake(char* pDevice)`
`611`	`611`	`const char* error_msg = sp_last_error_message();`
`612`	`612`	`if (error_msg)`
`613`	`613`	`{`
`614`		`- Log("libserialport error: %s\n", error_msg);`
	`614`	`+ Log("libserialport error: %s", error_msg);`
`615`	`615`	`}`
`616`	`616`	`}`
`617`	`617`	`std::this_thread::sleep_for(std::chrono::milliseconds(2));`
`@@ -692,7 +692,7 @@ bool ZeDMDComm::Handshake(char* pDevice)`
`692`	`692`	`const char* error_msg = sp_last_error_message();`
`693`	`693`	`if (error_msg)`
`694`	`694`	`{`
`695`		`- Log("libserialport error: %s\n", error_msg);`
	`695`	`+ Log("libserialport error: %s", error_msg);`
`696`	`696`	`}`
`697`	`697`	`}`
`698`	`698`
`@@ -706,7 +706,7 @@ bool ZeDMDComm::Handshake(char* pDevice)`
`706`	`706`	`const char* error_msg = sp_last_error_message();`
`707`	`707`	`if (error_msg)`
`708`	`708`	`{`
`709`		`- Log("libserialport error: %s\n", error_msg);`
	`709`	`+ Log("libserialport error: %s", error_msg);`
`710`	`710`	`}`
`711`	`711`	`}`
`712`	`712`
`@@ -876,7 +876,7 @@ bool ZeDMDComm::SendChunks(uint8_t* pData, uint16_t size)`
`876`	`876`	`const char* error_msg = sp_last_error_message();`
`877`	`877`	`if (error_msg)`
`878`	`878`	`{`
`879`		`- Log("libserialport error: %s\n", error_msg);`
	`879`	`+ Log("libserialport error: %s", error_msg);`
`880`	`880`	`}`
`881`	`881`	`}`
`882`	`882`	`}`
`@@ -919,7 +919,7 @@ bool ZeDMDComm::SendChunks(uint8_t* pData, uint16_t size)`
`919`	`919`	`const char* error_msg = sp_last_error_message();`
`920`	`920`	`if (error_msg)`
`921`	`921`	`{`
`922`		`- Log("libserialport error: %s\n", error_msg);`
	`922`	`+ Log("libserialport error: %s", error_msg);`
`923`	`923`	`}`
`924`	`924`	`}`
`925`	`925`
`@@ -955,7 +955,7 @@ bool ZeDMDComm::SendChunks(uint8_t* pData, uint16_t size)`
`955`	`955`	`const char* error_msg = sp_last_error_message();`
`956`	`956`	`if (error_msg)`
`957`	`957`	`{`
`958`		`- Log("libserialport error: %s\n", error_msg);`
	`958`	`+ Log("libserialport error: %s", error_msg);`
`959`	`959`	`}`
`960`	`960`	`}`
`961`	`961`