chore: Bump dependency versions (#11)

* chore: Bump dependency version

* chore: linter cleanup, remove some unnecessary parenthesis

* fix: silence a new clippy lint

* fix: remove an unused dep that is causing failures on osx

Cargo.toml

@@ -19,14 +19,13 @@ byteorder = "1.5"
 
 [dev-dependencies]
 criterion = { version = "0.5", features = ["html_reports"] }
-criterion-cycles-per-byte = "0.5"
 paste = "1.0"
 proptest = "1.0"
 rand = "0.8"
-test-strategy = "0.3"
+test-strategy = "0.4"
 refpack-sys = { git = "https://github.com/actioninja/refpack-sys.git" }
 ureq = "2.6"
-zip = "0.6"
+zip = "2.2"
 
 [[bench]]
 name = "synthetic_performance"

src/data/compression.rs

@@ -313,7 +313,7 @@ mod test {
     #[proptest]
     #[ignore]
     fn large_input_compression(
-        #[strategy(proptest::collection::vec(any::<u8>(), (100_000..=500_000)))] input: Vec<u8>,
+        #[strategy(proptest::collection::vec(any::<u8>(), 100_000..=500_000))] input: Vec<u8>,
     ) {
         let _unused = easy_compress::<Reference>(&input).unwrap();
     }

src/data/control/mod.rs

@@ -92,8 +92,8 @@ pub const LONG_LENGTH_MAX: u16 = 1_028;
 /// 1. mask first byte: `(byte0 & 0b0110_0000)` = `0PP0-0000`
 /// 2. shift left by 3: `(0PP0-0000 << 3)` = `0000-00PP--0000-0000`
 /// 3. OR with second:  `(0000-00PP--0000-0000 | 0000-0000--PPPP-PPPP)` =
-/// `0000-00PP--PPPP-PPPP` Another way to do this would be to first shift right
-/// by 5 and so on
+///    `0000-00PP--PPPP-PPPP` Another way to do this would be to first shift right
+///    by 5 and so on
 ///
 /// ## Key for description:
 /// - Len: Length of the command in bytes
@@ -142,10 +142,10 @@ pub const LONG_LENGTH_MAX: u16 = 1_028;
 /// We want to encode a literal length of 97
 ///
 /// 1. take `97 % 4` to get the "leftover" length - this will be used in next
-/// command following the literal
+///    command following the literal
 /// 2. take `(97 - 4) >> 2` to get the value to encode into the literal value
 /// 3. create a literal command with the result from 2, take that number of
-/// literals from the current literal buffer and write to stream
+///    literals from the current literal buffer and write to stream
 /// 4. in the next command, encode the leftover literal value from 1
 ///
 /// One extra unusual detail is that despite that it seems like te cap from the

src/data/mod.rs

@@ -167,7 +167,7 @@ mod test {
     ..Default::default()
     })]
     fn symmetrical_compression_large_input(
-        #[strategy(proptest::collection::vec(any::<u8>(), (2_000..=2_000)))] input: Vec<u8>,
+        #[strategy(proptest::collection::vec(any::<u8>(), 2_000..=2_000))] input: Vec<u8>,
     ) {
         let compressed = easy_compress::<Reference>(&input).unwrap();
         let decompressed = easy_decompress::<Reference>(&compressed).unwrap();

tests/fuzzers.rs

@@ -15,7 +15,7 @@ use test_strategy::proptest;
 
 #[proptest]
 fn rust_compression_symmetrical(
-    #[strategy(proptest::collection::vec(any::<u8>(), (100..=1_000)))] input: Vec<u8>,
+    #[strategy(proptest::collection::vec(any::<u8>(), 100..=1_000))] input: Vec<u8>,
 ) {
     let mut cloned = input.clone();
     let compressed = refpack_compress(&mut cloned);
@@ -27,7 +27,7 @@ fn rust_compression_symmetrical(
 
 #[proptest]
 fn rust_decompression_symmetrical(
-    #[strategy(proptest::collection::vec(any::<u8>(), (100..=1_000)))] input: Vec<u8>,
+    #[strategy(proptest::collection::vec(any::<u8>(), 100..=1_000))] input: Vec<u8>,
 ) {
     let mut compressed = easy_compress::<Reference>(&input).unwrap();