decoder::log: Rename PrettyLogger to StdoutLogger

CHANGELOG.md

@@ -7,6 +7,10 @@ and this project adheres to [Semantic Versioning](http://semver.org/).
 
 ## [Unreleased]
 
+- [#766] `decoder::log`: Rename `PrettyLogger` to `StdoutLogger`
+
+[#766]: https://github.com/knurling-rs/defmt/pull/766
+
 ## [v0.3.5] - 2023-06-19
 
 - [#760]: `defmt-macros`: Upgrade to syn 2

decoder/src/log/json_logger.rs

@@ -4,11 +4,11 @@ use time::OffsetDateTime;
 
 use std::io::{self, Write};
 
-use super::{pretty_logger::PrettyLogger, DefmtRecord};
+use super::{DefmtRecord, StdoutLogger};
 
 pub(crate) struct JsonLogger {
  should_log: Box<dyn Fn(&Metadata) -> bool + Sync + Send>,
- host_logger: PrettyLogger,
+ host_logger: StdoutLogger,
 }
 
 impl Log for JsonLogger {
@@ -44,7 +44,7 @@ impl JsonLogger {
  pub fn new(should_log: impl Fn(&Metadata) -> bool + Sync + Send + 'static) -> Box<Self> {
  Box::new(Self {
  should_log: Box::new(should_log),
- host_logger: PrettyLogger::new_unboxed(true, |_| true),
+ host_logger: StdoutLogger::new_unboxed(true, |_| true),
  })
  }
 

decoder/src/log/mod.rs

@@ -7,14 +7,14 @@
 //! [`defmt`]: https://crates.io/crates/defmt
 
 mod json_logger;
-mod pretty_logger;
+mod stdout_logger;
 
 use log::{Level, LevelFilter, Metadata, Record};
 use serde::{Deserialize, Serialize};
 
 use std::fmt;
 
-use self::{json_logger::JsonLogger, pretty_logger::PrettyLogger};
+use self::{json_logger::JsonLogger, stdout_logger::StdoutLogger};
 use crate::Frame;
 
 const DEFMT_TARGET_MARKER: &str = "defmt@";
@@ -128,7 +128,7 @@ pub fn init_logger(
  should_log: impl Fn(&Metadata) -> bool + Sync + Send + 'static,
 ) {
  log::set_boxed_logger(match json {
- false => PrettyLogger::new(always_include_location, should_log),
+ false => StdoutLogger::new(always_include_location, should_log),
  true => {
  JsonLogger::print_schema_version();
  JsonLogger::new(should_log)

decoder/src/log/pretty_logger.rs → decoder/src/log/stdout_logger.rs

@@ -10,15 +10,15 @@ use std::{
 
 use super::DefmtRecord;
 
-pub(crate) struct PrettyLogger {
+pub(crate) struct StdoutLogger {
  always_include_location: bool,
  should_log: Box<dyn Fn(&Metadata) -> bool + Sync + Send>,
  /// Number of characters used by the timestamp. This may increase over time and is used to align
  /// messages.
  timing_align: AtomicUsize,
 }
 
-impl Log for PrettyLogger {
+impl Log for StdoutLogger {
  fn enabled(&self, metadata: &Metadata) -> bool {
  (self.should_log)(metadata)
  }
@@ -49,7 +49,7 @@ impl Log for PrettyLogger {
  fn flush(&self) {}
 }
 
-impl PrettyLogger {
+impl StdoutLogger {
  pub fn new(
  always_include_location: bool,
  should_log: impl Fn(&Metadata) -> bool + Sync + Send + 'static,