Clean up casts and DMX config guards

Improve type-safety and warning hygiene in low-level helpers: `debug::PrintBits` now uses explicit unsigned casts with matching format specifiers, `inet_aton` is modernized (`nullptr`, scoped union alias, clearer declarations/casts), and `inet_ntoa` uses `auto*` for the byte pointer. In `displayudf.h`, DMX-related feature macros are reordered so `DMX_MAX_PORTS` is undefined before DMX node output headers are included, ensuring consistent configuration for relevant output modes. Copyright year ranges were also updated.

Author: vanvught

common/include/firmware/debug/debug_printbits.h

@@ -26,21 +26,21 @@
 #ifndef COMMON_DEBUG_DEBUG_PRINTBITS_H_
 #define COMMON_DEBUG_DEBUG_PRINTBITS_H_
 
-#include <cstdio>
 #include <cstdint>
 
 namespace debug {
 #ifdef NDEBUG
 inline void PrintBits([[maybe_unused]] uint32_t u) {}
 #else
+#include <cstdio>
 inline void PrintBits(uint32_t u) {
-    printf("%.8x ", u);
+    printf("%.8x ", static_cast<unsigned>(u));
     uint32_t b = 1U << 31;
 
     for (uint32_t i = 0; i < 32; i++) {
         if ((b & u) == b) {
             uint32_t bit_number = 31 - i;
-            printf("%-2d ", bit_number);
+            printf("%-2u ", static_cast<unsigned>(bit_number));
         }
         b = b >> 1;
     }
@@ -50,4 +50,4 @@ inline void PrintBits(uint32_t u) {
 #endif
 } // namespace debug
 
-#endif /* COMMON_DEBUG_DEBUG_PRINTBITS_H_ */
+#endif // COMMON_DEBUG_DEBUG_PRINTBITS_H_

lib-clib/src/inet_aton.cpp

@@ -2,7 +2,7 @@
  * @file inet_aton.cpp
  *
  */
-/* Copyright (C) 2016-2025 by Arjan van Vught mailto:info@gd32-dmx.org
+/* Copyright (C) 2016-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
@@ -27,14 +27,18 @@
 #include <cctype>
 #include <netinet/in.h>
 
-typedef union pcast32 {
+namespace {
+using _pcast32 = union pcast32 {
     uint32_t u32;
     uint8_t u8[4];
-} _pcast32;
+};
+} // namespace
 
 extern "C" int inet_aton(const char* cp, struct in_addr* ip_address) {
     const char* b = cp;
-    int i, j, k;
+    int i;
+    int j;
+    int k;
     _pcast32 cast32;
 
     for (i = 0; i < 3; i++) {
@@ -46,12 +50,12 @@ extern "C" int inet_aton(const char* cp, struct in_addr* ip_address) {
                 return 0;
             }
 
-            if (0 == isdigit((int)*b)) {
+            if (0 == isdigit(static_cast<int>(*b))) {
                 return 0;
             }
 
             j++;
-            k = k * 10 + (int)*b - (int)'0';
+            k = (k * 10) + static_cast<int>(*b) - '0';
             b++;
         }
 
@@ -72,13 +76,13 @@ extern "C" int inet_aton(const char* cp, struct in_addr* ip_address) {
         }
 
         j++;
-        k = k * 10 + (int)*b - (int)'0';
+        k = (k * 10) + static_cast<int>(*b) - '0';
         b++;
     }
 
     cast32.u8[i] = (uint8_t)k;
 
-    if (ip_address != 0) {
+    if (ip_address != nullptr) {
         ip_address->s_addr = cast32.u32;
     }
 

lib-clib/src/inet_ntoa.cpp

@@ -29,7 +29,7 @@
 static char buffer[18];
 
 extern "C" char* inet_ntoa(struct in_addr in) {
-    unsigned char* bytes = (unsigned char*)∈
+    auto* bytes = (unsigned char*)∈
     snprintf(buffer, sizeof(buffer), "%d.%d.%d.%d", bytes[0], bytes[1], bytes[2], bytes[3]);
     return buffer;
 }

lib-displayudf/include/displayudf.h

@@ -2,7 +2,7 @@
  * @file displayudf.h
  *
  */
-/* Copyright (C) 2019-2025 by Arjan van Vught mailto:info@gd32-dmx.org
+/* Copyright (C) 2019-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
@@ -51,13 +51,16 @@
 #if defined(RDM_RESPONDER)
 #include "rdmdeviceresponder.h"
 #endif
+#if defined(RDM_RESPONDER) || defined(OUTPUT_DMX_MONITOR) || defined(OUTPUT_DMX_PCA9685) || defined(OUTPUT_DMX_PIXEL) || defined(OUTPUT_DMX_TLC59711)
+#define HAVE_DMX_START_ADDRESS
+#endif
+#if defined(RDM_RESPONDER) || defined(OUTPUT_DMX_MONITOR) || defined(OUTPUT_DMX_PCA9685) || defined(OUTPUT_DMX_TLC59711)
+#undef DMX_MAX_PORTS
+#endif
 #include "dmxnode_outputtype.h"
 #if defined(DMXNODE_OUTPUT_DMX)
 #include "dmx.h"
 #endif
-#if defined(RDM_RESPONDER) || defined(OUTPUT_DMX_MONITOR) || defined(OUTPUT_DMX_PCA9685) || defined(OUTPUT_DMX_PIXEL) || defined(OUTPUT_DMX_TLC59711)
-#define HAVE_DMX_START_ADDRESS
-#endif
 
 namespace displayudf {
 inline constexpr uint32_t kLabelMaxRows = 6;

lib-dmxnode/src/json/json_config_dmxnode.cpp

@@ -1,7 +1,7 @@
 /**
  * @file json_config_dmxnode.cpp
  */
-/* Copyright (C) 2025 by Arjan van Vught mailto:info@gd32-dmx.org
+/* Copyright (C) 2025-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