Refactor: use declarative macro to implement formats #67
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #67 +/- ##
==========================================
- Coverage 81.57% 76.41% -5.16%
==========================================
Files 12 12
Lines 266 229 -37
==========================================
- Hits 217 175 -42
- Misses 49 54 +5 ☔ View full report in Codecov by Sentry. |
| crate::send::compression::Format::Gzip, | ||
| flate2::write::GzEncoder::new, | ||
| flate2::read::MultiGzDecoder::new, | ||
| std::io::Read | Send, |
There was a problem hiding this comment.
This | is a bit weird, I wanted + but that is not available syntax for a path
More info: https://internals.rust-lang.org/t/allow-to-follow-path-fragments-in-declarative-macros/13676
| gz, | ||
| "gz", |
| level: Level, | ||
| ) -> Result<Box<dyn io::Write + Send + 'a>, Error> { | ||
| Ok(Box::new( | ||
| zstd::stream::write::Encoder::new(out, level.into())?.auto_finish(), |
There was a problem hiding this comment.
Couldn't do zstd because it has the extra .auto_finish(), need to look more into it
| cfg_if! { | ||
| if #[cfg(feature = "zstd")] { |
There was a problem hiding this comment.
Turns out we don't need cfg_if!, #[cfg(feature = "zstd")] and #[cfg(not(feature = "zstd"))] are enough for our use case
| compression::Format::Zstd => compression::new_zstd_encoder(out_stream, level), | ||
| compression::Format::Gzip => compression::gz::encoder(out_stream, level), |
There was a problem hiding this comment.
now each format is a submodule instead of a function, and all submodules have an encoder and a decoder function. These are all private, so it doesn't break the public API
|
Very good idea and work, if you need/want help just ask. |
Trying out a declarative macro to reduce copy-and-paste between
basicandsendmodules.