commit 479933181ebdc39b6c41cf567d4cb52393ebeffd
author Austin Schuh <austin.linux@gmail.com> Tue Jul 20 18:06:38 2021 -0700
committer Gerrit Code Review <gerrit@software.frc971.org> Tue Jul 20 18:06:38 2021 -0700
tree 72e774fce7452dc792fe716f94b71a52f0478f20
parent 1b6c38dfb30336779fa051f162c05a5973c6580a
parent b7c8d2a619666f7dfe712afb6c99cc186b61a25e

Merge "Track our time buffer more explicitly"

aos/events/logging/logfile_utils.cc

diff --git a/aos/events/logging/logfile_utils.cc b/aos/events/logging/logfile_utils.cc
index e9d0dd3..7600c7e 100644
--- a/aos/events/logging/logfile_utils.cc
+++ b/aos/events/logging/logfile_utils.cc
@@ -909,6 +909,8 @@
     CHECK(timestamp_mapper->nodes_data_[node()].peer == nullptr);
 
     timestamp_mapper->nodes_data_[node()].peer = this;
+
+    node_data->save_for_peer = true;
   }
 }
 
@@ -1199,6 +1201,7 @@
   }
   for (NodeData &node_data : nodes_data_) {
     if (!node_data.any_delivered) continue;
+    if (!node_data.save_for_peer) continue;
     if (node_data.channels[m->channel_index].delivered) {
       // TODO(austin): This copies the data...  Probably not worth stressing
       // about yet.

aos/events/logging/logfile_utils.h

diff --git a/aos/events/logging/logfile_utils.h b/aos/events/logging/logfile_utils.h
index f876d25..93d9e8b 100644
--- a/aos/events/logging/logfile_utils.h
+++ b/aos/events/logging/logfile_utils.h
@@ -633,6 +633,9 @@
     // bools in delivered below are true.
     bool any_delivered = false;
 
+    // True if we have a peer and therefore should be saving data for it.
+    bool save_for_peer = false;
+
     // Peer pointer.  This node is only to be considered if a peer is set.
     TimestampMapper *peer = nullptr;