Fix format specifier for printing CAN message ID in CanMsg.h.

haydenroche5 · haydenroche5 · commit 42098e28a2f4 · 2023-08-25T10:57:36.000-07:00
Regarding the X format specifier: C standard §7.21.6.1: "The unsigned int
argument is converted to ..." On some platforms (e.g. mbed_nano), uint32_t,
which is what's being printed here, is an unsigned long int, so this code causes
a compiler warning. The fix is to use the format specifiers §7.8.1, specifically
PRIX32.
diff --git a/api/CanMsg.h b/api/CanMsg.h
@@ -12,7 +12,7 @@
  * INCLUDE
  **************************************************************************************/
 
-#include <stdint.h>
+#include <inttypes.h>
 #include <string.h>
 
 #include <Arduino.h>
@@ -68,7 +68,7 @@ class CanMsg : public Printable
     size_t len = 0;
 
     /* Print the header. */
-    len = snprintf(buf, sizeof(buf), "[%08X] (%d) : ", id, data_length);
+    len = snprintf(buf, sizeof(buf), "[%08" PRIX32 "] (%d) : ", id, data_length);
     size_t n = p.write(buf, len);
 
     /* Print the data. */
