Normalize console error messages and fixes

Append trailing newlines to diagnostic console::Error calls across multiple modules (malloc, software timers, DHCP, IGMP, TCP, UDP) for consistent output formatting. Add console::Write declaration and replace console::Puts(": ") with console::Write(": ", 2) in perror to use a write-with-length API. Add missing <cstdint> include in UART console implementation and bump copyright year in perror.cpp.

Author: vanvught

lib-clib/src/malloc.cpp

@@ -150,7 +150,7 @@ extern "C"
 
             if (next > block_limit)
             {
-                console::Error("malloc: out of memory");
+                console::Error("malloc: out of memory\n");
 #ifdef DEBUG_HEAP
                 debug_heap();
 #endif

lib-clib/src/perror.cpp

@@ -2,7 +2,7 @@
  * @file perror.cpp
  *
  */
-/* Copyright (C) 2020-2025 by Arjan van Vught mailto:info@gd32-dmx.org
+/* Copyright (C) 2020-2026 by Arjan van Vught mailto:info@gd32-dmx.org
  *
  * Permission is hereby granted, free of charge, to any person obtaining a copy
  * of this software and associated documentation files (the "Software"), to deal
@@ -31,6 +31,7 @@ namespace console
 void Error(const char*);
 int Putc(int);
 int Puts(const char*);
+void Write(const char*, unsigned int);
 } // namespace console
 
 /*
@@ -107,7 +108,7 @@ extern "C"
         if (s && *s)
         {
             console::Error(s);
-            console::Puts(": ");
+            console::Write(": ", 2);
         }
 
         console::Error(ptr);

lib-hal/console/uart0/console.cpp

@@ -23,6 +23,7 @@
  * THE SOFTWARE.
  */
 
+#include <cstdint>
 #include <cstring>
 
 #include "console.h"

lib-hal/superloop/softwaretimers.cpp

@@ -77,7 +77,7 @@ TimerHandle_t SoftwareTimerAdd(uint32_t interval_millis, const TimerCallbackFunc
     if (s_timers_count >= hal::kSoftwareTimersMax)
     {
 #ifndef NDEBUG
-        console::Error("SoftwareTimerAdd: Max timer limit reached");
+        console::Error("SoftwareTimerAdd: Max timer limit reached\n");
 #endif
         return -1;
     }
@@ -134,7 +134,7 @@ bool SoftwareTimerDelete(TimerHandle_t& id)
     }
 
 #ifndef NDEBUG
-    console::Error("SoftwareTimerDelete: Timer not found");
+    console::Error("SoftwareTimerDelete: Timer not found\n");
 #endif
 
     DEBUG_EXIT();
@@ -163,7 +163,7 @@ bool SoftwareTimerChange(TimerHandle_t id, uint32_t interval_millis)
     }
 
 #ifndef NDEBUG
-    console::Error("SoftwareTimerChange: Timer not found");
+    console::Error("SoftwareTimerChange: Timer not found\n");
 #endif
 
     return false;

lib-network/src/core/ipv4/dhcp.cpp

@@ -250,7 +250,7 @@ void Inform()
 #ifndef NDEBUG
     if (kHandle < 0)
     {
-        console::Error("DHCP Inform");
+        console::Error("DHCP Inform\n");
         return;
     }
 #endif
@@ -766,7 +766,7 @@ bool Start()
 #ifndef NDEBUG
     if (dhcp->handle < 0)
     {
-        console::Error("DHCP Start");
+        console::Error("DHCP Start\n");
         DEBUG_EXIT();
         return false;
     }

lib-network/src/core/ipv4/igmp.cpp

@@ -378,7 +378,7 @@ void static Join(uint32_t group_address)
     }
 
 #ifndef NDEBUG
-    console::Error("igmp::Join");
+    console::Error("igmp::Join\n");
 #endif
     DEBUG_ENTRY();
 }

lib-network/src/core/tcp.cpp

@@ -1664,7 +1664,7 @@ int32_t Connect(uint32_t remote_ip, uint16_t remote_port, CallbackConnect cb_con
     const auto& netif = netif::global::netif_default;
     if (__builtin_expect((netif.ip.addr == 0), 0))
     {
-        console::Error("Connect: No ip!");
+        console::Error("Connect: No ip!\n");
         return -1;
     }
 
@@ -1673,7 +1673,7 @@ int32_t Connect(uint32_t remote_ip, uint16_t remote_port, CallbackConnect cb_con
     auto* tcb = AllocTcb(remote_port, &out_index);
     if (tcb == nullptr)
     {
-        console::Error("Connect: No TCB!");
+        console::Error("Connect: No TCB!\n");
         return -2;
     }
 
@@ -1714,15 +1714,15 @@ int32_t Close(ConnHandle conn_handle) // graceful FIN
 {
     if (conn_handle >= TCP_MAX_TCBS_ALLOWED)
     {
-        console::Error("Close: Connection handle!");
+        console::Error("Close: Connection handle!\n");
         return -1;
     }
 
     auto* c = &s_tcbs[conn_handle];
 
     if (!c->in_use || c->state == kStateClosed)
     {
-        console::Error("Close: TCB!");
+        console::Error("Close: TCB!\n");
         return -1;
     }
 
@@ -1743,7 +1743,7 @@ int32_t Close(ConnHandle conn_handle) // graceful FIN
     // We only support graceful close from states where FIN makes sense here.
     if (c->state != kStateEstablished && c->state != kStateCloseWait)
     {
-        console::Error("Close: Not graceful!");
+        console::Error("Close: Not graceful!\n");
         return -1;
     }
 

lib-network/src/core/udp.cpp

@@ -249,7 +249,7 @@ int32_t Begin(uint16_t localport, UdpCallbackFunctionPtr callback)
     }
 
 #ifndef NDEBUG
-    console::Error("network::udp::Begin");
+    console::Error("network::udp::Begin\n");
 #endif
     return -1;
 }
@@ -274,7 +274,7 @@ int32_t End(uint16_t localport)
     }
 
 #ifndef NDEBUG
-    console::Error("network::udp::End");
+    console::Error("network::udp::End\n");
 #endif
     return -1;
 }