🚨 Fix compiler warnings in printf formatting

Author: paulreimer

actor_model/src/process.cpp

@@ -124,7 +124,7 @@ auto Process::send(const Message& message)
     ESP_LOGE(
       get_uuid_str(pid).c_str(),
       "Unable to send message (payload size %zu)",
-      message.payload()->size()
+      static_cast<size_t>(message.payload()->size())
     );
   }
   return did_send;

firmware_update/src/firmware_update_actor.cpp

@@ -667,7 +667,7 @@ auto firmware_update_actor_behaviour(
 
           if (all_files_valid and (new_version > current_version))
           {
-            printf("Newer firmware update version %d available, downloading\n", new_version);
+            printf("Newer firmware update version %lu available, downloading\n", new_version);
 
             if (firmware_update_metadata->checksum())
             {

network_manager/src/network_check_actor.cpp

@@ -62,7 +62,7 @@ auto ping_success_callback(esp_ping_handle_t hdl, void *args)
     sizeof(elapsed_time)
   );
   printf(
-    "%d bytes from %s icmp_seq=%d ttl=%d time=%d ms\n",
+    "%lu bytes from %s icmp_seq=%d ttl=%d time=%lu ms\n",
     recv_len,
     inet_ntoa(target_addr.u_addr.ip4),
     seqno,
@@ -98,7 +98,7 @@ auto ping_end_callback(esp_ping_handle_t hdl, void *args)
   } else {
       printf("\n--- %s ping statistics ---\n", inet6_ntoa(*ip_2_ip6(&target_addr)));
   }
-  printf("%d packets transmitted, %d received, %d%% packet loss, time %dms\n",
+  printf("%lu packets transmitted, %lu received, %lu%% packet loss, time %lums\n",
           transmitted, received, loss, total_time_ms);
   // delete the ping sessions, so that we clean up all resources and can create a new ping session
   // we don't have to call delete function in the callback, instead we can call delete function from other tasks

requests/src/queued_endpoint_actor.cpp

@@ -250,7 +250,7 @@ auto queued_endpoint_actor_behaviour(
           TAG,
           "Response error (%d), resending: '%.*s'\n",
           response->code(),
-          response->body()->size(),
+          static_cast<int>(response->body()->size()),
           response->body()->data()
         );
 

uuid/lib/sole/sole.hpp

@@ -176,27 +176,27 @@ namespace sole {
         auto i = 0;
 
         // Include 1 byte for -, and 1 for null (ignored)
-        snprintf(&s[i], 8 + 1 + 1, "%08x-", (a));
+        snprintf(&s[i], 8 + 1 + 1, "%08lx-", (a));
         i += 8 + 1;
 
         // Include 1 byte for -, and 1 for null (ignored)
-        snprintf(&s[i], 4 + 1 + 1, "%04x-", (b >> 16));
+        snprintf(&s[i], 4 + 1 + 1, "%04lx-", (b >> 16));
         i += 4 + 1;
 
         // Include 1 byte for -, and 1 for null (ignored)
-        snprintf(&s[i], 4 + 1 + 1, "%04x-", (b & 0xFFFF));
+        snprintf(&s[i], 4 + 1 + 1, "%04lx-", (b & 0xFFFF));
         i += 4 + 1;
 
         // Include 1 byte for -, and 1 for null (ignored)
-        snprintf(&s[i], 4 + 1 + 1, "%04x-", (c >> 16 ));
+        snprintf(&s[i], 4 + 1 + 1, "%04lx-", (c >> 16 ));
         i += 4 + 1;
 
         // Include 1 byte for null (ignored)
-        snprintf(&s[i], 4 + 1, "%04x", (c & 0xFFFF));
+        snprintf(&s[i], 4 + 1, "%04lx", (c & 0xFFFF));
         i += 4;
 
         // Include 1 byte for null (ignored)
-        snprintf(&s[i], 8 + 1, "%08x", (d));
+        snprintf(&s[i], 8 + 1, "%08lx", (d));
         i += 8;
 
         // Strip the trailing null that snprintf adds