From be62f065a62048b798e8bb2b8e3699bdd5b6d517 Mon Sep 17 00:00:00 2001 From: zirkonya Date: Tue, 1 Sep 2026 09:51:18 +0200 Subject: add proc macros ; benchmark ; example --- src/codec/decode.rs | 251 ++++++++++++++++++++------------------- src/codec/encode.rs | 336 +++++++++++++++++++++++++++++----------------------- 2 files changed, 319 insertions(+), 268 deletions(-) (limited to 'src/codec') diff --git a/src/codec/decode.rs b/src/codec/decode.rs index 529c448..d068760 100644 --- a/src/codec/decode.rs +++ b/src/codec/decode.rs @@ -1,150 +1,133 @@ -//! Decoding: reading protocol values from a byte stream. use std::{ collections::{BTreeMap, BTreeSet, BinaryHeap, HashMap, HashSet}, hash::Hash, - io::Read, sync::Arc, }; -use crate::{DEFAULT_BUFFER_LEN, codec::error::Result}; -use crate::{codec::error::CodecError, context::Context}; +use crate::{codec::error::Result, codec::error::CodecError, context::Context}; macro_rules! impl_decode { - (u8) => { - impl Decode for u8 { - fn decode(reader: &mut dyn Read, _: &Context) -> Result + ($t:ty) => { + impl Decode for $t { + fn decode(buf: &mut &[u8], _: &Context) -> Result where Self: Sized, { - let mut byte = [0u8; 1]; - reader.read_exact(&mut byte)?; - Ok(byte[0]) + const BYTES: usize = std::mem::size_of::<$t>(); + if buf.len() < BYTES { + return Err(CodecError::IoError(std::io::Error::new( + std::io::ErrorKind::UnexpectedEof, + "insufficient bytes", + ))); + } + let (bytes, rest) = buf.split_at(BYTES); + *buf = rest; + Ok(<$t>::from_be_bytes(bytes.try_into().unwrap())) } - fn decode_slice(reader: &mut dyn Read, _: &Context) -> Result> + fn decode_slice(buf: &mut &[u8], _: &Context) -> Result> where Self: Sized, { - let mut buf = Vec::new(); - reader.read_to_end(&mut buf)?; - Ok(buf) + const BYTES: usize = std::mem::size_of::<$t>(); + let count = buf.len() / BYTES; + let mut vec = Vec::with_capacity(count); + for _ in 0..count { + let (bytes, rest) = buf.split_at(BYTES); + *buf = rest; + vec.push(<$t>::from_be_bytes(bytes.try_into().unwrap())); + } + Ok(vec) } } }; - ($type: ty) => { - impl Decode for $type { - // decode number using big endian - fn decode(reader: &mut dyn Read, _: &Context) -> Result - where - Self: Sized, - { - const BYTES: usize = (<$type>::BITS / 8) as usize; - let mut bytes = [0; BYTES]; - reader.read_exact(&mut bytes)?; - Ok(<$type>::from_be_bytes(bytes)) - } +} - fn decode_slice(reader: &mut dyn Read, _: &Context) -> Result> +macro_rules! impl_decode_tuples { + ($($generic: ident),+) => { + impl Decode for ($($generic,)+) + where + $($generic: Decode),+ + { + #[allow(non_snake_case)] + fn decode(buf: &mut &[u8], ctx: &Context) -> Result where Self: Sized, { - let mut buf = Vec::with_capacity(DEFAULT_BUFFER_LEN); - reader.read_to_end(&mut buf); - const BYTES: usize = <$type>::BITS as usize / 8; - Ok(buf - .chunks(BYTES) - .map(|slice| { - let Some(bytes): Option<&[u8; BYTES]> = slice.as_array() else { - unreachable!() - }; - <$type>::from_be_bytes(*bytes) - }) - .collect()) + $( + let $generic = $generic::decode(buf, ctx)?; + )+ + Ok(($($generic,)+)) } } }; } -macro_rules! impl_decode_tuples { - ($($generic: ident),+) => { - impl Decode for ($($generic,)+) where $($generic: Decode,)+ { - #[allow(non_snake_case)] - fn decode(reader: &mut dyn Read, ctx: &Context) -> Result - where - Self: Sized { - $( - let $generic = $generic::decode(reader, ctx)?; - )+ - Ok(($($generic,)+)) - } - } - }; -} - -/// Read a value from a byte stream +/// Zero-copy decoding: reads directly from a byte slice pub trait Decode { - fn decode(reader: &mut dyn Read, ctx: &Context) -> Result + fn decode(buf: &mut &[u8], ctx: &Context) -> Result where Self: Sized; - // TODO : change for anything else than Vec - /// assume reader contains only the slice to decode - fn decode_slice(reader: &mut dyn Read, ctx: &Context) -> Result> + fn decode_slice(buf: &mut &[u8], ctx: &Context) -> Result> where Self: Sized, { - let mut buf = Vec::new(); - loop { - match Self::decode(reader, ctx) { - Ok(value) => buf.push(value), + let mut vec = Vec::new(); + while !buf.is_empty() { + match Self::decode(buf, ctx) { + Ok(value) => vec.push(value), Err(CodecError::IoError(err)) - if let std::io::ErrorKind::UnexpectedEof = err.kind() => + if err.kind() == std::io::ErrorKind::UnexpectedEof => { - return Ok(buf); + return Ok(vec); } Err(err) => return Err(err), } } + Ok(vec) } } -// ~ Decode arrays +// ── Decode arrays ─────────────────────────────────────────────────────── impl> Decode for Vec { - fn decode(reader: &mut dyn Read, ctx: &Context) -> Result + fn decode(buf: &mut &[u8], ctx: &Context) -> Result where Self: Sized, { - let len = usize::decode(reader, ctx)?; + let len = usize::decode(buf, ctx)?; let mut vec = Vec::with_capacity(len); for _ in 0..len { - vec.push(T::decode(reader, ctx)?); + vec.push(T::decode(buf, ctx)?); } Ok(vec) } } impl> Decode for Option { - /// Assume is always some - fn decode(reader: &mut dyn Read, ctx: &Context) -> Result + fn decode(buf: &mut &[u8], ctx: &Context) -> Result where Self: Sized, { - T::decode(reader, ctx).map(Some) + if buf.is_empty() { + return Ok(None); + } + T::decode(buf, ctx).map(Some) } } -// ~ Decode slices +// ── Decode slices ─────────────────────────────────────────────────────── impl Decode for [T; S] where T: Decode + Default + Copy, { - fn decode(reader: &mut dyn Read, ctx: &Context) -> Result + fn decode(buf: &mut &[u8], ctx: &Context) -> Result where Self: Sized, { - let slice = T::decode_slice(reader, ctx)?; + let slice = T::decode_slice(buf, ctx)?; let got = slice.len(); slice .as_array() @@ -157,7 +140,7 @@ impl Decode for &[T] where T: Decode + Default + Copy, { - fn decode(_: &mut dyn Read, _: &Context) -> Result + fn decode(_: &mut &[u8], _: &Context) -> Result where Self: Sized, { @@ -165,18 +148,22 @@ where } } -// ~ Decode set +// ── Decode set ────────────────────────────────────────────────────────── impl Decode for HashSet where T: Decode + Eq + Hash, { - fn decode(reader: &mut dyn Read, ctx: &Context) -> Result + fn decode(buf: &mut &[u8], ctx: &Context) -> Result where Self: Sized, { - let slice = T::decode_slice(reader, ctx)?; - Ok(slice.into_iter().collect()) + let len = usize::decode(buf, ctx)?; + let mut set = HashSet::with_capacity(len); + for _ in 0..len { + set.insert(T::decode(buf, ctx)?); + } + Ok(set) } } @@ -184,43 +171,57 @@ impl Decode for BTreeSet where T: Decode + Ord, { - fn decode(reader: &mut dyn Read, ctx: &Context) -> Result + fn decode(buf: &mut &[u8], ctx: &Context) -> Result where Self: Sized, { - let slice = T::decode_slice(reader, ctx)?; - Ok(slice.into_iter().collect()) + let len = usize::decode(buf, ctx)?; + let mut set = BTreeSet::new(); + for _ in 0..len { + set.insert(T::decode(buf, ctx)?); + } + Ok(set) } } -// ~ Decode misc +// ── Decode misc ───────────────────────────────────────────────────────── impl Decode for BinaryHeap where T: Decode + Ord, { - fn decode(reader: &mut dyn Read, ctx: &Context) -> Result + fn decode(buf: &mut &[u8], ctx: &Context) -> Result where Self: Sized, { - let slice = T::decode_slice(reader, ctx)?; - Ok(BinaryHeap::from_iter(slice)) + let len = usize::decode(buf, ctx)?; + let mut heap = BinaryHeap::with_capacity(len); + for _ in 0..len { + heap.push(T::decode(buf, ctx)?); + } + Ok(heap) } } -// ~ Decode maps +// ── Decode maps ───────────────────────────────────────────────────────── impl Decode for HashMap where K: Decode + Eq + Hash, V: Decode, { - fn decode(reader: &mut dyn Read, ctx: &Context) -> Result + fn decode(buf: &mut &[u8], ctx: &Context) -> Result where Self: Sized, { - let slice = <(K, V) as Decode>::decode_slice(reader, ctx)?; - Ok(slice.into_iter().collect()) + let len = usize::decode(buf, ctx)?; + let mut map = HashMap::with_capacity(len); + for _ in 0..len { + let key = K::decode(buf, ctx)?; + let val = V::decode(buf, ctx)?; + map.insert(key, val); + } + Ok(map) } } @@ -229,19 +230,25 @@ where K: Decode + Ord, V: Decode, { - fn decode(reader: &mut dyn Read, ctx: &Context) -> Result + fn decode(buf: &mut &[u8], ctx: &Context) -> Result where Self: Sized, { - let slice = <(K, V) as Decode>::decode_slice(reader, ctx)?; - Ok(slice.into_iter().collect()) + let len = usize::decode(buf, ctx)?; + let mut map = BTreeMap::new(); + for _ in 0..len { + let key = K::decode(buf, ctx)?; + let val = V::decode(buf, ctx)?; + map.insert(key, val); + } + Ok(map) } } -// ~ Decode string +// ── Decode string - requires length prefix ────────────────────────────── impl Decode for &str { - fn decode(_: &mut dyn Read, _: &Context) -> Result + fn decode(_: &mut &[u8], _: &Context) -> Result where Self: Sized, { @@ -250,17 +257,17 @@ impl Decode for &str { } impl Decode for String { - fn decode(reader: &mut dyn Read, _: &Context) -> Result + fn decode(buf: &mut &[u8], ctx: &Context) -> Result where Self: Sized, { - let mut bytes: Vec = Vec::new(); - reader.read_to_end(&mut bytes)?; - Ok(String::from_utf8_lossy(&bytes).to_string()) + // String must be length-prefixed - decode as LenPrefixed + crate::types::prefix::length::LenPrefixed::::decode(buf, ctx) + .map(|p| p.data().clone()) } } -// ~ Decode primitive +// ── Decode primitive ──────────────────────────────────────────────────── impl_decode!(u8); impl_decode!(u16); @@ -277,39 +284,43 @@ impl_decode!(i128); impl_decode!(isize); impl Decode for f64 { - fn decode(reader: &mut dyn Read, ctx: &Context) -> Result + fn decode(buf: &mut &[u8], ctx: &Context) -> Result where Self: Sized, { - let bits = u64::decode(reader, ctx)?; - let value = f64::from_bits(bits); - Ok(value) + let bits = u64::decode(buf, ctx)?; + Ok(f64::from_bits(bits)) } } impl Decode for f32 { - fn decode(reader: &mut dyn Read, ctx: &Context) -> Result + fn decode(buf: &mut &[u8], ctx: &Context) -> Result where Self: Sized, { - let bits = u32::decode(reader, ctx)?; - let value = f32::from_bits(bits); - Ok(value) + let bits = u32::decode(buf, ctx)?; + Ok(f32::from_bits(bits)) } } impl Decode for bool { - fn decode(reader: &mut dyn Read, _: &Context) -> Result + fn decode(buf: &mut &[u8], _: &Context) -> Result where Self: Sized, { - let mut byte = [0_u8]; - reader.read_exact(&mut byte)?; - Ok(byte[0] != 0) + if buf.is_empty() { + return Err(CodecError::IoError(std::io::Error::new( + std::io::ErrorKind::UnexpectedEof, + "insufficient bytes for bool", + ))); + } + let b = buf[0]; + *buf = &buf[1..]; + Ok(b != 0) } } -// ~ Decode tuple +// ── Decode tuple ──────────────────────────────────────────────────────── impl_decode_tuples!(A); impl_decode_tuples!(A, B); @@ -329,14 +340,14 @@ impl_decode_tuples!(A, B, C, D, E, F, G, H, I, J, K, L, M, N, O); impl_decode_tuples!(A, B, C, D, E, F, G, H, I, J, K, L, M, N, O, P); impl_decode_tuples!(A, B, C, D, E, F, G, H, I, J, K, L, M, N, O, P, Q); -// ~ Decode pointer +// ── Decode pointer ────────────────────────────────────────────────────── impl Decode for std::sync::Arc where T: Decode, { - fn decode(reader: &mut dyn Read, ctx: &Context) -> Result { - T::decode(reader, ctx).map(Arc::new) + fn decode(buf: &mut &[u8], ctx: &Context) -> Result { + T::decode(buf, ctx).map(Arc::new) } } @@ -344,11 +355,11 @@ impl Decode for Arc<[T]> where T: Decode, { - fn decode(reader: &mut dyn Read, ctx: &Context) -> Result + fn decode(buf: &mut &[u8], ctx: &Context) -> Result where Self: Sized, { - let slice = T::decode_slice(reader, ctx)?; + let slice = T::decode_slice(buf, ctx)?; Ok(slice.into()) } } diff --git a/src/codec/encode.rs b/src/codec/encode.rs index 7463995..735aa70 100644 --- a/src/codec/encode.rs +++ b/src/codec/encode.rs @@ -1,49 +1,36 @@ -//! Encoding: writing protocol values into a byte stream. use crate::codec::error::Result; use crate::context::Context; -use std::{ - collections::{BTreeMap, BTreeSet, BinaryHeap, HashMap, HashSet, LinkedList, VecDeque}, - io::Write, -}; +use std::collections::{BTreeMap, BTreeSet, BinaryHeap, HashMap, HashSet, LinkedList, VecDeque}; macro_rules! impl_encode { - (u8) => { - impl Encode for u8 { - /// write number using big endian - fn encode(&self, buffer: &mut dyn Write, _: &Context) -> Result { - let size = buffer.write(&self.to_be_bytes())?; - Ok(size) + ($t:ty) => { + impl Encode for $t { + fn encode(&self, buffer: &mut Vec, _: &Context) -> Result { + buffer.extend_from_slice(&self.to_be_bytes()); + Ok(std::mem::size_of::<$t>()) } - /// write raw bytes - fn encode_slice( - slice: &[Self], - buffer: &mut dyn Write, - _: &Context, - ) -> Result { - buffer.write(slice).map_err(Into::into) - } - } - }; - ($t: ty) => { - impl Encode for $t { - /// write number using big endian - fn encode(&self, buffer: &mut dyn Write, _: &Context) -> Result { - let size = buffer.write(&self.to_be_bytes())?; - Ok(size) + fn encode_slice(slice: &[Self], buffer: &mut Vec, _: &Context) -> Result + where + Self: Sized, + { + let len = slice.len() * std::mem::size_of::<$t>(); + buffer.reserve(len); + for item in slice { + buffer.extend_from_slice(&item.to_be_bytes()); + } + Ok(len) } - fn encode_slice( - slice: &[Self], - buffer: &mut dyn Write, - _: &Context, - ) -> Result { - let len = buffer.write( - &slice - .iter() - .flat_map(|n| n.to_be_bytes()) - .collect::>(), - )?; + fn encode_iter<'a, I>(iter: I, buffer: &mut Vec, ctx: &Context) -> Result + where + Self: 'a + Sized, + I: IntoIterator, + { + let mut len = 0; + for item in iter { + len += item.encode(buffer, ctx)?; + } Ok(len) } } @@ -57,7 +44,7 @@ macro_rules! impl_encode_tuples { $($generic: Encode),+ { #[allow(non_snake_case)] - fn encode(&self, buffer: &mut dyn Write, ctx: &Context) -> Result { + fn encode(&self, buffer: &mut Vec, ctx: &Context) -> Result { let ($($generic,)+): &($($generic,)+) = self; let mut len = 0; $(len += $generic.encode(buffer, ctx)?;)+ @@ -67,10 +54,11 @@ macro_rules! impl_encode_tuples { }; } -/// Write a value into a byte stream +/// Zero-copy encoding: writes directly into a byte buffer pub trait Encode { - fn encode(&self, buffer: &mut dyn Write, ctx: &Context) -> Result; - fn encode_slice(slice: &[Self], buffer: &mut dyn Write, ctx: &Context) -> Result + fn encode(&self, buffer: &mut Vec, ctx: &Context) -> Result; + + fn encode_slice(slice: &[Self], buffer: &mut Vec, ctx: &Context) -> Result where Self: Sized, { @@ -80,204 +68,257 @@ pub trait Encode { } Ok(len) } + + fn encode_iter<'a, I>(iter: I, buffer: &mut Vec, ctx: &Context) -> Result + where + Self: 'a + Sized, + I: IntoIterator, + { + let mut len = 0; + for item in iter { + len += item.encode(buffer, ctx)?; + } + Ok(len) + } } -// ~ Encode arrays +// ── Encode arrays ─────────────────────────────────────────────────────── impl> Encode for Vec { - /// Encode each element of Vec - /// use `CountPrefix` to prefix the vector with number of elements - /// use `LenPrefix` to prefix the vector with encoded byte size - fn encode(&self, buffer: &mut dyn Write, ctx: &Context) -> Result { + fn encode(&self, buffer: &mut Vec, ctx: &Context) -> Result { T::encode_slice(self, buffer, ctx) } } impl> Encode for VecDeque { - /// Encode each element of VecDeque - /// use `CountPrefix` to prefix the VecDeque with number of elements - /// use `LenPrefix` to prefix the VecDeque with encoded byte size - fn encode(&self, buffer: &mut dyn Write, ctx: &Context) -> Result { - let (front, _) = self.as_slices(); - T::encode_slice(front, buffer, ctx) + fn encode(&self, buffer: &mut Vec, ctx: &Context) -> Result { + let (front, back) = self.as_slices(); + let mut len = 0; + len += T::encode_slice(front, buffer, ctx)?; + len += T::encode_slice(back, buffer, ctx)?; + Ok(len) } } -impl> Encode for LinkedList -where - T: Clone, -{ - /// Encode each element of LinkedList (use clone..) - /// use `CountPrefix` to prefix the LinkedList with number of elements - /// use `LenPrefix` to prefix the LinkedList with encoded byte size - fn encode(&self, buffer: &mut dyn Write, ctx: &Context) -> Result { - let mut view = Vec::with_capacity(self.len()); - view.extend(self.iter().cloned()); - T::encode_slice(&view, buffer, ctx) +impl> Encode for LinkedList { + fn encode(&self, buffer: &mut Vec, ctx: &Context) -> Result { + let mut len = 0; + for item in self { + len += item.encode(buffer, ctx)?; + } + Ok(len) } } -// ~ Encode slices +// ── Encode slices ─────────────────────────────────────────────────────── impl> Encode for &[T] { - /// Encode each element of slice (use clone..) - /// use `CountPrefix` to prefix the slice with number of elements - /// use `LenPrefix` to prefix the slice with encoded byte size - fn encode(&self, buffer: &mut dyn Write, ctx: &Context) -> Result { + fn encode(&self, buffer: &mut Vec, ctx: &Context) -> Result { T::encode_slice(self, buffer, ctx) } } impl, const S: usize> Encode for [T; S] { - /// Encode each element of slice (use clone..) - /// use `CountPrefix` to prefix the slice with number of elements - /// use `LenPrefix` to prefix the slice with encoded byte size - fn encode(&self, buffer: &mut dyn Write, ctx: &Context) -> Result { + fn encode(&self, buffer: &mut Vec, ctx: &Context) -> Result { T::encode_slice(self, buffer, ctx) } } impl> Encode for [T] { - /// Encode each element of slice (use clone..) - /// use `CountPrefix` to prefix the slice with number of elements - /// use `LenPrefix` to prefix the slice with encoded byte size - fn encode(&self, buffer: &mut dyn Write, ctx: &Context) -> Result { + fn encode(&self, buffer: &mut Vec, ctx: &Context) -> Result { + T::encode_slice(self, buffer, ctx) + } +} + +// ── Encode references to collections ──────────────────────────────────── + +impl> Encode for &Vec { + fn encode(&self, buffer: &mut Vec, ctx: &Context) -> Result { T::encode_slice(self, buffer, ctx) } } -// ~ Encode set +impl> Encode for &VecDeque { + fn encode(&self, buffer: &mut Vec, ctx: &Context) -> Result { + let (front, back) = self.as_slices(); + let mut len = 0; + len += T::encode_slice(front, buffer, ctx)?; + len += T::encode_slice(back, buffer, ctx)?; + Ok(len) + } +} -impl> Encode for HashSet +impl> Encode for &LinkedList { + fn encode(&self, buffer: &mut Vec, ctx: &Context) -> Result { + let mut len = 0; + for item in (*self).iter() { + len += item.encode(buffer, ctx)?; + } + Ok(len) + } +} + +impl> Encode for &HashSet { + fn encode(&self, buffer: &mut Vec, ctx: &Context) -> Result { + T::encode_iter(self.iter(), buffer, ctx) + } +} + +impl> Encode for &BTreeSet { + fn encode(&self, buffer: &mut Vec, ctx: &Context) -> Result { + T::encode_iter(self.iter(), buffer, ctx) + } +} + +impl> Encode for &BinaryHeap { + fn encode(&self, buffer: &mut Vec, ctx: &Context) -> Result { + T::encode_slice(self.as_slice(), buffer, ctx) + } +} + +impl Encode for &HashMap where - T: Clone, + K: Encode, + V: Encode, { - /// Encode each element of HashSet (use clone..) - /// use `CountPrefix` to prefix the HashSet with number of elements - /// use `LenPrefix` to prefix the HashSet with encoded byte size - fn encode(&self, buffer: &mut dyn Write, ctx: &Context) -> Result { - let mut view = Vec::with_capacity(self.len()); - view.extend(self.iter().cloned()); - T::encode_slice(&view, buffer, ctx) + fn encode(&self, buffer: &mut Vec, ctx: &Context) -> Result { + let mut len = 0; + for (k, v) in (*self).iter() { + len += k.encode(buffer, ctx)?; + len += v.encode(buffer, ctx)?; + } + Ok(len) } } -impl> Encode for BTreeSet +impl Encode for &BTreeMap where - T: Clone, + K: Encode, + V: Encode, { - /// Encode each element of BTreeSet (use clone..) - /// use `CountPrefix` to prefix the BTreeSet with number of elements - /// use `LenPrefix` to prefix the BTreeSet with encoded byte size - fn encode(&self, buffer: &mut dyn Write, ctx: &Context) -> Result { - let mut view = Vec::with_capacity(self.len()); - view.extend(self.iter().cloned()); - T::encode_slice(&view, buffer, ctx) + fn encode(&self, buffer: &mut Vec, ctx: &Context) -> Result { + let mut len = 0; + for (k, v) in (*self).iter() { + len += k.encode(buffer, ctx)?; + len += v.encode(buffer, ctx)?; + } + Ok(len) } } -// ~ Encode misc +// ── Encode set (owned) ────────────────────────────────────────────────── + +impl> Encode for HashSet { + fn encode(&self, buffer: &mut Vec, ctx: &Context) -> Result { + T::encode_iter(self.iter(), buffer, ctx) + } +} + +impl> Encode for BTreeSet { + fn encode(&self, buffer: &mut Vec, ctx: &Context) -> Result { + T::encode_iter(self.iter(), buffer, ctx) + } +} + +// ── Encode misc ───────────────────────────────────────────────────────── impl Encode for BinaryHeap where T: Encode, { - fn encode(&self, buffer: &mut dyn Write, ctx: &Context) -> Result { + fn encode(&self, buffer: &mut Vec, ctx: &Context) -> Result { T::encode_slice(self.as_slice(), buffer, ctx) } } -// ~ Encode maps +// ── Encode maps (owned) ───────────────────────────────────────────────── impl Encode for HashMap where - K: Encode + Clone, - V: Encode + Clone, + K: Encode, + V: Encode, { - /// Encode each element of HashMap (use clone..) - /// use `CountPrefix` to prefix the HashMap with number of pairs - /// use `LenPrefix` to prefix the HashMap with encoded byte size - fn encode(&self, buffer: &mut dyn Write, ctx: &Context) -> Result { - let slice: Vec<(K, V)> = self.clone().into_iter().collect(); - <(K, V) as Encode>::encode_slice(&slice, buffer, ctx) + fn encode(&self, buffer: &mut Vec, ctx: &Context) -> Result { + let mut len = 0; + for (k, v) in self { + len += k.encode(buffer, ctx)?; + len += v.encode(buffer, ctx)?; + } + Ok(len) } } impl Encode for BTreeMap where - K: Encode + Clone, - V: Encode + Clone, + K: Encode, + V: Encode, { - /// Encode each element of BTreeMap (use clone..) - /// use `CountPrefix` to prefix the BTreeMap with number of pairs - /// use `LenPrefix` to prefix the BTreeMap with encoded byte size - fn encode(&self, buffer: &mut dyn Write, ctx: &Context) -> Result { - let slice: Vec<(K, V)> = self.clone().into_iter().collect(); - <(K, V) as Encode>::encode_slice(&slice, buffer, ctx) + fn encode(&self, buffer: &mut Vec, ctx: &Context) -> Result { + let mut len = 0; + for (k, v) in self { + len += k.encode(buffer, ctx)?; + len += v.encode(buffer, ctx)?; + } + Ok(len) } } -// ~ Encode string +// ── Encode string ─────────────────────────────────────────────────────── + impl Encode for String { - /// Encode the string using utf8 - /// use `CountPrefix` to prefix the string with char length - /// use `LenPrefix` to prefix the string with utf8 bytes length - fn encode(&self, buffer: &mut dyn Write, _: &Context) -> Result { + fn encode(&self, buffer: &mut Vec, _: &Context) -> Result { let utf8 = self.as_bytes(); - buffer.write_all(utf8)?; + buffer.extend_from_slice(utf8); Ok(utf8.len()) } } impl Encode for &str { - /// Encode the string using utf8 - /// use `CountPrefix` to prefix the string with char length - /// use `LenPrefix` to prefix the string with utf8 bytes length - fn encode(&self, buffer: &mut dyn Write, _: &Context) -> Result { + fn encode(&self, buffer: &mut Vec, _: &Context) -> Result { let utf8 = self.as_bytes(); - buffer.write_all(utf8)?; + buffer.extend_from_slice(utf8); Ok(utf8.len()) } } impl> Encode for Option { - /// Encode `T` if Some(T) or do nothing if None - fn encode(&self, buffer: &mut dyn Write, ctx: &Context) -> Result { + fn encode(&self, buffer: &mut Vec, ctx: &Context) -> Result { match self { - Some(val) => { - let size = val.encode(buffer, ctx)?; - Ok(size) - } + Some(val) => val.encode(buffer, ctx), None => Ok(0), } } } -// ~ Encode primitive +// ── Encode primitive ──────────────────────────────────────────────────── + impl Encode for bool { - fn encode(&self, buffer: &mut dyn Write, _: &Context) -> Result { - let size = buffer.write(&[*self as u8])?; - Ok(size) + fn encode(&self, buffer: &mut Vec, _: &Context) -> Result { + buffer.push(*self as u8); + Ok(1) } - fn encode_slice(slice: &[Self], buffer: &mut dyn Write, _: &Context) -> Result + fn encode_slice(slice: &[Self], buffer: &mut Vec, _: &Context) -> Result where Self: Sized, { - let len = buffer.write(&slice.iter().map(|n| *n as u8).collect::>())?; + let len = slice.len(); + buffer.reserve(len); + for &b in slice { + buffer.push(b as u8); + } Ok(len) } } -// ~ Encode pointer +// ── Encode pointer ────────────────────────────────────────────────────── impl Encode for std::sync::Arc where T: Encode, { - fn encode(&self, buffer: &mut dyn Write, ctx: &Context) -> Result { - let data = self.as_ref(); - data.encode(buffer, ctx) + fn encode(&self, buffer: &mut Vec, ctx: &Context) -> Result { + self.as_ref().encode(buffer, ctx) } } @@ -285,9 +326,8 @@ impl Encode for std::sync::Arc<[T]> where T: Encode, { - fn encode(&self, buffer: &mut dyn Write, ctx: &Context) -> Result { - let data = self.as_ref(); - data.encode(buffer, ctx) + fn encode(&self, buffer: &mut Vec, ctx: &Context) -> Result { + self.as_ref().encode(buffer, ctx) } } @@ -312,14 +352,14 @@ impl_encode!(f64); #[cfg(feature = "f128")] impl_encode!(f128); -// ~ Encode tuple +// ── Encode tuple ──────────────────────────────────────────────────────── impl Encode for () { - fn encode(&self, _: &mut dyn Write, _: &Context) -> Result { + fn encode(&self, _: &mut Vec, _: &Context) -> Result { Ok(0) } - fn encode_slice(_: &[Self], _: &mut dyn Write, _: &Context) -> Result + fn encode_slice(_: &[Self], _: &mut Vec, _: &Context) -> Result where Self: Sized, { -- cgit v1.2.3