Fix defmt losing bytes

CHANGELOG.md

@@ -16,6 +16,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ### Fixed
 
 - Fixed printing panic backtraces when using `esp-println` and `defmt` (#496)
+- Fixed defmt parsing when data is read in parts (#503)
 
 ### Changed
 

espflash/src/cli/monitor/mod.rs

@@ -99,6 +99,11 @@ pub fn monitor_with(
     let stdout = stdout();
     let mut stdout = ResolvingPrinter::new(elf, stdout.lock());
 
+    let mut parser: Box<dyn InputParser> = match log_format {
+        LogFormat::Defmt => Box::new(parser::esp_defmt::EspDefmt::new(elf)),
+        LogFormat::Serial => Box::new(parser::serial::Serial),
+    };
+
     let mut buff = [0; 1024];
     loop {
         let read_count = match serial.serial_port_mut().read(&mut buff) {
@@ -108,16 +113,7 @@ pub fn monitor_with(
             err => err,
         }?;
 
-        match log_format {
-            LogFormat::Defmt => {
-                let mut parser = parser::esp_defmt::EspDefmt::new(elf);
-                parser.feed(&buff[0..read_count], &mut stdout);
-            }
-            LogFormat::Serial => {
-                let mut parser = parser::serial::Serial;
-                parser.feed(&buff[0..read_count], &mut stdout);
-            }
-        }
+        parser.feed(&buff[0..read_count], &mut stdout);
 
         // Don't forget to flush the writer!
         stdout.flush().ok();

espflash/src/cli/monitor/parser/esp_defmt.rs

@@ -106,11 +106,11 @@ impl EspDefmt {
         }
     }
 
-    fn handle_raw(bytes: &[u8], out: &mut impl Write) {
+    fn handle_raw(bytes: &[u8], out: &mut dyn Write) {
         out.write_all(bytes).unwrap();
     }
 
-    fn handle_defmt(frame: Frame<'_>, out: &mut impl Write) {
+    fn handle_defmt(frame: Frame<'_>, out: &mut dyn Write) {
         match frame.level() {
             Some(level) => {
                 let color = match level {
@@ -142,7 +142,7 @@ impl EspDefmt {
 }
 
 impl InputParser for EspDefmt {
-    fn feed(&mut self, bytes: &[u8], out: &mut impl Write) {
+    fn feed(&mut self, bytes: &[u8], out: &mut dyn Write) {
         let Some(table) = self.table.as_mut() else {
             Self::handle_raw(bytes, out);
             return;

espflash/src/cli/monitor/parser/mod.rs

@@ -13,7 +13,7 @@ use regex::Regex;
 use crate::cli::monitor::{line_endings::normalized, symbols::Symbols};
 
 pub trait InputParser {
-    fn feed(&mut self, bytes: &[u8], out: &mut impl Write);
+    fn feed(&mut self, bytes: &[u8], out: &mut dyn Write);
 }
 
 // Pattern to much a function address in serial output.

espflash/src/cli/monitor/parser/serial.rs

@@ -5,7 +5,7 @@ use crate::cli::monitor::parser::InputParser;
 pub struct Serial;
 
 impl InputParser for Serial {
-    fn feed(&mut self, bytes: &[u8], out: &mut impl Write) {
+    fn feed(&mut self, bytes: &[u8], out: &mut dyn Write) {
         out.write_all(bytes).unwrap();
     }
 }