Fix presubmit build errors

This change combines the following existing fixes:
- fix ArrowKeyNavigationMenu member initialization order (change 55425)
- use %zu for printing size_t (change 56510)
- fix comparison of different signedness (change 56509)

Change-Id: I4538a7a7fad028c0eaef4cb77f4a877487b61c7a
Signed-off-by: Sui Chen <suichen6@gmail.com>
diff --git a/dbus_capture.cpp b/dbus_capture.cpp
index 715fd41..50ed37f 100644
--- a/dbus_capture.cpp
+++ b/dbus_capture.cpp
@@ -14,8 +14,8 @@
 
 #include "analyzer.hpp"
 #include "histogram.hpp"
-#include "sensorhelper.hpp"
 #include "main.hpp"
+#include "sensorhelper.hpp"
 
 #include <ncurses.h>
 #include <stdlib.h>
@@ -30,11 +30,11 @@
 
 namespace dbus_top_analyzer
 {
-    extern DBusTopStatistics g_dbus_statistics;
-    extern Histogram<float> g_mc_time_histogram;
+extern DBusTopStatistics g_dbus_statistics;
+extern Histogram<float> g_mc_time_histogram;
 } // namespace dbus_top_analyzer
 
-static void TrackMessage(sd_bus_message* m)
+static void TrackMessage([[maybe_unused]] sd_bus_message* m)
 {}
 // Obtain a Monitoring DBus connection
 int AcquireBus(sd_bus** ret)
@@ -188,7 +188,7 @@
             }
             // This is for the bottom-right window
             dbus_top_analyzer::g_dbus_statistics.OnNewDBusMessage(
-            sender_uniq.c_str(), dest_uniq.c_str(), interface, path, member,
+                sender_uniq.c_str(), dest_uniq.c_str(), interface, path, member,
                 type, m);
             sd_bus_message_unref(m);
         }