commit 15c22b1895c13234d6e84f96dcc4c4dab68cf952
author Alexei Strots <alexei.strots@bluerivertech.com> Tue Apr 04 16:27:17 2023 -0700
committer Austin Schuh <austin.schuh@bluerivertech.com> Fri May 05 18:16:55 2023 -0700
tree 8279932dee72576f6ed241d3c8ec721ab1a8648e
parent bc082d82b0bf3690605c77a48ab5a40dad99a2a8

Remove DetachedBufferFileWriter and use FileBackend

DetachedBufferFileWriter was a temp solution. Making file via log
backend is a better one.

Change-Id: I0c212f404f5a3f70034e674c697b5ddcc18614e6
Signed-off-by: Austin Schuh <austin.schuh@bluerivertech.com>

aos/events/logging/log_edit.cc

diff --git a/aos/events/logging/log_edit.cc b/aos/events/logging/log_edit.cc
index 55b7666..7e822ca 100644
--- a/aos/events/logging/log_edit.cc
+++ b/aos/events/logging/log_edit.cc
@@ -48,12 +48,13 @@
     aos::logger::SpanReader span_reader(orig_path);
     CHECK(!span_reader.ReadMessage().empty()) << ": Empty header, aborting";
 
-    aos::logger::DetachedBufferFileWriter buffer_writer(
-        FLAGS_logfile,
+    aos::logger::FileBackend file_backend("/");
+    aos::logger::DetachedBufferWriter buffer_writer(
+        file_backend.RequestFile(FLAGS_logfile),
         std::make_unique<aos::logger::DummyEncoder>(FLAGS_max_message_size));
     {
-      aos::logger::DataEncoder::SpanCopier coppier(header.span());
-      buffer_writer.CopyMessage(&coppier, aos::monotonic_clock::min_time);
+      aos::logger::DataEncoder::SpanCopier copier(header.span());
+      buffer_writer.CopyMessage(&copier, aos::monotonic_clock::min_time);
     }
 
     while (true) {
@@ -63,8 +64,8 @@
       }
 
       {
-        aos::logger::DataEncoder::SpanCopier coppier(msg_data);
-        buffer_writer.CopyMessage(&coppier, aos::monotonic_clock::min_time);
+        aos::logger::DataEncoder::SpanCopier copier(msg_data);
+        buffer_writer.CopyMessage(&copier, aos::monotonic_clock::min_time);
       }
     }
   } else {