From 683205ad0333ce36206397c438f7d251ffc47b56 Mon Sep 17 00:00:00 2001 From: "Carol (Nichols || Goulding)" Date: Fri, 19 Jun 2020 15:23:15 -0400 Subject: [PATCH] refactor: Use `Vec::clear` instead of `Vec::truncate(0)` --- benches/encoders.rs | 2 +- delorean_tsm/src/encoders/float.rs | 2 +- delorean_tsm/src/encoders/integer.rs | 2 +- delorean_tsm/src/encoders/string.rs | 2 +- delorean_tsm/src/encoders/timestamp.rs | 2 +- 5 files changed, 5 insertions(+), 5 deletions(-) diff --git a/benches/encoders.rs b/benches/encoders.rs index b0a86cd8e6..d03f4adcf2 100644 --- a/benches/encoders.rs +++ b/benches/encoders.rs @@ -75,7 +75,7 @@ fn benchmark_decode( |b, &decoded_len| { let mut decoded_mut = Vec::with_capacity(decoded_len); b.iter(|| { - decoded_mut.truncate(0); + decoded_mut.clear(); decode(&encoded, &mut decoded_mut).unwrap(); }); }, diff --git a/delorean_tsm/src/encoders/float.rs b/delorean_tsm/src/encoders/float.rs index 99e4bfd727..94876d7d1a 100644 --- a/delorean_tsm/src/encoders/float.rs +++ b/delorean_tsm/src/encoders/float.rs @@ -21,7 +21,7 @@ fn is_sentinel_u64(v: u64, sentinel: u64) -> bool { /// representations based on those are stored. #[allow(clippy::many_single_char_names)] pub fn encode(src: &[f64], dst: &mut Vec) -> Result<(), Box> { - dst.truncate(0); // reset buffer. + dst.clear(); // reset buffer. if src.is_empty() { return Ok(()); } diff --git a/delorean_tsm/src/encoders/integer.rs b/delorean_tsm/src/encoders/integer.rs index 2861ced0df..7060c26c85 100644 --- a/delorean_tsm/src/encoders/integer.rs +++ b/delorean_tsm/src/encoders/integer.rs @@ -16,7 +16,7 @@ enum Encoding { /// further compressed if possible, either via bit-packing using simple8b or by /// run-length encoding the deltas if they're all the same. pub fn encode(src: &[i64], dst: &mut Vec) -> Result<(), Box> { - dst.truncate(0); // reset buffer. + dst.clear(); // reset buffer. if src.is_empty() { return Ok(()); } diff --git a/delorean_tsm/src/encoders/string.rs b/delorean_tsm/src/encoders/string.rs index 66c29c9d48..bef7b1fae9 100644 --- a/delorean_tsm/src/encoders/string.rs +++ b/delorean_tsm/src/encoders/string.rs @@ -12,7 +12,7 @@ const MAX_I32: usize = i32::MAX as usize; /// Encodes a slice of byte slices representing string data into a vector of bytes. Currently uses /// Snappy compression. pub fn encode(src: &[&[u8]], dst: &mut Vec) -> Result<(), Box> { - dst.truncate(0); // reset buffer + dst.clear(); // reset buffer if src.is_empty() { return Ok(()); } diff --git a/delorean_tsm/src/encoders/timestamp.rs b/delorean_tsm/src/encoders/timestamp.rs index fe8a9fe9ac..0899f6abe5 100644 --- a/delorean_tsm/src/encoders/timestamp.rs +++ b/delorean_tsm/src/encoders/timestamp.rs @@ -17,7 +17,7 @@ enum Encoding { /// encoded using RLE. If not, as long as the deltas are not bigger than simple8b::MAX_VALUE /// they can be encoded using simple8b. pub fn encode(src: &[i64], dst: &mut Vec) -> Result<(), Box> { - dst.truncate(0); // reset buffer. + dst.clear(); // reset buffer. if src.is_empty() { return Ok(()); }