refactor: Pull out writer as a top-level concept

In thinking over #365 and looking over `Logger`, this is an orthogonal,
top-level concept

Maybe at a later point, we can consider if we should revisit how this is
exposed in the API.

Author: epage

src/fmt/mod.rs

@@ -71,7 +71,6 @@ use log::Record;
 mod humantime;
 #[cfg(feature = "kv")]
 mod kv;
-pub(crate) mod writer;
 
 #[cfg(feature = "color")]
 pub use anstyle as style;
@@ -80,10 +79,10 @@ pub use anstyle as style;
 pub use self::humantime::Timestamp;
 #[cfg(feature = "kv")]
 pub use self::kv::*;
-pub use self::writer::Target;
-pub use self::writer::WriteStyle;
+pub use crate::writer::Target;
+pub use crate::writer::WriteStyle;
 
-use self::writer::{Buffer, Writer};
+use crate::writer::{Buffer, Writer};
 
 /// Formatting precision of timestamps.
 ///
@@ -650,7 +649,7 @@ mod tests {
     }
 
     fn formatter() -> Formatter {
-        let writer = writer::Builder::new()
+        let writer = crate::writer::Builder::new()
             .write_style(WriteStyle::Never)
             .build();
 

src/lib.rs

@@ -265,6 +265,7 @@
 #![warn(clippy::print_stdout)]
 
 mod logger;
+mod writer;
 
 pub mod fmt;
 

src/logger.rs

@@ -3,8 +3,8 @@ use std::{borrow::Cow, cell::RefCell, env, io};
 use log::{LevelFilter, Log, Metadata, Record, SetLoggerError};
 
 use crate::fmt;
-use crate::fmt::writer::{self, Writer};
 use crate::fmt::{FormatFn, Formatter};
+use crate::writer::{self, Writer};
 
 /// The default name for the environment variable to read filters from.
 pub const DEFAULT_FILTER_ENV: &str = "RUST_LOG";

src/fmt/writer/buffer.rs src/writer/buffer.rssrc/fmt/writer/buffer.rs renamed to src/writer/buffer.rs

@@ -1,6 +1,6 @@
 use std::{io, sync::Mutex};
 
-use crate::fmt::writer::WriteStyle;
+use crate::writer::WriteStyle;
 
 #[derive(Debug)]
 pub(crate) struct BufferWriter {

src/fmt/writer/mod.rs src/writer/mod.rssrc/fmt/writer/mod.rs renamed to src/writer/mod.rs

@@ -1,10 +1,11 @@
 mod buffer;
 mod target;
 
-use self::buffer::BufferWriter;
 use std::{io, mem, sync::Mutex};
 
-pub(crate) use self::buffer::Buffer;
+use buffer::BufferWriter;
+
+pub(crate) use buffer::Buffer;
 
 pub use target::Target;