commit 99f7c6a79a63b99943ebb7fca8ac96f90a6a60a4
author Austin Schuh <austin.linux@gmail.com> Tue Jun 25 22:07:44 2024 -0700
committer Austin Schuh <austin.linux@gmail.com> Sat Jul 20 11:40:35 2024 -0700
tree f62d61eae9cc18132596df7f52d6ec28d09dc46a
parent 6927bc31c854f264ff7d431918b008620b6b4712

Move over to ABSL logging and flags.

Removes gperftools too since that wants gflags.

Here come the fireworks.

Change-Id: I79cb7bcf60f1047fbfa28bfffc21a0fd692e4b1c
Signed-off-by: Austin Schuh <austin.linux@gmail.com>

aos/network/timestamp_channel.cc

diff --git a/aos/network/timestamp_channel.cc b/aos/network/timestamp_channel.cc
index 778c2e7..1a821c5 100644
--- a/aos/network/timestamp_channel.cc
+++ b/aos/network/timestamp_channel.cc
@@ -1,14 +1,15 @@
 #include "aos/network/timestamp_channel.h"
 
+#include "absl/flags/flag.h"
 #include "absl/strings/str_cat.h"
 
-DEFINE_bool(combined_timestamp_channel_fallback, true,
-            "If true, fall back to using the combined timestamp channel if the "
-            "single timestamp channel doesn't exist for a timestamp.");
-DEFINE_bool(check_timestamp_channel_frequencies, true,
-            "If true, include a debug CHECK to ensure that remote timestamp "
-            "channels are configured to have at least as great a frequency as "
-            "the corresponding data channel.");
+ABSL_FLAG(bool, combined_timestamp_channel_fallback, true,
+          "If true, fall back to using the combined timestamp channel if the "
+          "single timestamp channel doesn't exist for a timestamp.");
+ABSL_FLAG(bool, check_timestamp_channel_frequencies, true,
+          "If true, include a debug CHECK to ensure that remote timestamp "
+          "channels are configured to have at least as great a frequency as "
+          "the corresponding data channel.");
 
 namespace aos::message_bridge {
 
@@ -55,7 +56,7 @@
     return split_timestamp_channel;
   }
 
-  if (!FLAGS_combined_timestamp_channel_fallback) {
+  if (!absl::GetFlag(FLAGS_combined_timestamp_channel_fallback)) {
     LOG(FATAL) << "Failed to find new timestamp channel {\"name\": \""
                << split_timestamp_channel_name << "\", \"type\": \""
                << RemoteMessage::GetFullyQualifiedName() << "\"} for "
@@ -112,7 +113,7 @@
 
   const Channel *timestamp_channel = finder.ForChannel(channel, connection);
 
-  if (FLAGS_check_timestamp_channel_frequencies) {
+  if (absl::GetFlag(FLAGS_check_timestamp_channel_frequencies)) {
     // Sanity-check that the timestamp channel can actually support full-rate
     // messages coming through on the source channel.
     CHECK_GE(timestamp_channel->frequency(), channel->frequency())