Merge pull request #286 from quantatic/main

Use explicit paths rather than glob imports for zstd wrapper types.

Author: robjtede

src/zstd.rs

@@ -1,8 +1,5 @@
 //! This module contains zstd-specific types for async-compression.
 
-use libzstd::stream::raw::CParameter::*;
-use libzstd::stream::raw::DParameter::*;
-
 /// A compression parameter for zstd. This is a stable wrapper around zstd's own `CParameter`
 /// type, to abstract over different versions of the zstd library.
 ///
@@ -14,74 +11,74 @@ pub struct CParameter(libzstd::stream::raw::CParameter);
 impl CParameter {
     /// Window size in bytes (as a power of two)
     pub fn window_log(value: u32) -> Self {
-        Self(WindowLog(value))
+        Self(libzstd::stream::raw::CParameter::WindowLog(value))
     }
 
     /// Size of the initial probe table in 4-byte entries (as a power of two)
     pub fn hash_log(value: u32) -> Self {
-        Self(HashLog(value))
+        Self(libzstd::stream::raw::CParameter::HashLog(value))
     }
 
     /// Size of the multi-probe table in 4-byte entries (as a power of two)
     pub fn chain_log(value: u32) -> Self {
-        Self(ChainLog(value))
+        Self(libzstd::stream::raw::CParameter::ChainLog(value))
     }
 
     /// Number of search attempts (as a power of two)
     pub fn search_log(value: u32) -> Self {
-        Self(SearchLog(value))
+        Self(libzstd::stream::raw::CParameter::SearchLog(value))
     }
 
     /// Minimum size of matches searched for
     pub fn min_match(value: u32) -> Self {
-        Self(MinMatch(value))
+        Self(libzstd::stream::raw::CParameter::MinMatch(value))
     }
 
     /// Strategy-dependent length modifier
     pub fn target_length(value: u32) -> Self {
-        Self(TargetLength(value))
+        Self(libzstd::stream::raw::CParameter::TargetLength(value))
     }
 
     /// Enable long-distance matching mode to look for and emit long-distance references.
     ///
     /// This increases the default window size.
     pub fn enable_long_distance_matching(value: bool) -> Self {
-        Self(EnableLongDistanceMatching(value))
+        Self(libzstd::stream::raw::CParameter::EnableLongDistanceMatching(value))
     }
 
     /// Size of the long-distance matching table (as a power of two)
     pub fn ldm_hash_log(value: u32) -> Self {
-        Self(LdmHashLog(value))
+        Self(libzstd::stream::raw::CParameter::LdmHashLog(value))
     }
 
     /// Minimum size of long-distance matches searched for
     pub fn ldm_min_match(value: u32) -> Self {
-        Self(LdmMinMatch(value))
+        Self(libzstd::stream::raw::CParameter::LdmMinMatch(value))
     }
 
     /// Size of each bucket in the LDM hash table for collision resolution (as a power of two)
     pub fn ldm_bucket_size_log(value: u32) -> Self {
-        Self(LdmBucketSizeLog(value))
+        Self(libzstd::stream::raw::CParameter::LdmBucketSizeLog(value))
     }
 
     /// Frequency of using the LDM hash table (as a power of two)
     pub fn ldm_hash_rate_log(value: u32) -> Self {
-        Self(LdmHashRateLog(value))
+        Self(libzstd::stream::raw::CParameter::LdmHashRateLog(value))
     }
 
     /// Emit the size of the content (default: true).
     pub fn content_size_flag(value: bool) -> Self {
-        Self(ContentSizeFlag(value))
+        Self(libzstd::stream::raw::CParameter::ContentSizeFlag(value))
     }
 
     /// Emit a checksum (default: false).
     pub fn checksum_flag(value: bool) -> Self {
-        Self(ChecksumFlag(value))
+        Self(libzstd::stream::raw::CParameter::ChecksumFlag(value))
     }
 
     /// Emit a dictionary ID when using a custom dictionary (default: true).
     pub fn dict_id_flag(value: bool) -> Self {
-        Self(DictIdFlag(value))
+        Self(libzstd::stream::raw::CParameter::DictIdFlag(value))
     }
 
     /// Number of threads to spawn.
@@ -97,14 +94,14 @@ impl CParameter {
     // TODO: make this a normal feature guarded fn on next breaking release
     #[cfg_attr(docsrs, doc(cfg(feature = "zstdmt")))]
     pub fn nb_workers(value: u32) -> Self {
-        Self(NbWorkers(value))
+        Self(libzstd::stream::raw::CParameter::NbWorkers(value))
     }
 
     /// Number of bytes given to each worker.
     ///
     /// If set to 0, zstd selects a job size based on compression parameters.
     pub fn job_size(value: u32) -> Self {
-        Self(JobSize(value))
+        Self(libzstd::stream::raw::CParameter::JobSize(value))
     }
 
     pub(crate) fn as_zstd(&self) -> libzstd::stream::raw::CParameter {
@@ -123,7 +120,7 @@ pub struct DParameter(libzstd::stream::raw::DParameter);
 impl DParameter {
     /// Maximum window size in bytes (as a power of two)
     pub fn window_log_max(value: u32) -> Self {
-        Self(WindowLogMax(value))
+        Self(libzstd::stream::raw::DParameter::WindowLogMax(value))
     }
 
     pub(crate) fn as_zstd(&self) -> libzstd::stream::raw::DParameter {