From e88686e4f46e301114aba692364a90838a4beba6 Mon Sep 17 00:00:00 2001 From: Paul Hauner Date: Wed, 19 Sep 2018 17:48:52 +1000 Subject: [PATCH] Remove Result from list encoding, panic instead --- ssz/src/encode.rs | 11 ++--------- 1 file changed, 2 insertions(+), 9 deletions(-) diff --git a/ssz/src/encode.rs b/ssz/src/encode.rs index d5095af6b..5080924cc 100644 --- a/ssz/src/encode.rs +++ b/ssz/src/encode.rs @@ -60,20 +60,13 @@ impl SszStream { /// The length of the list will be concatenated to the stream, then /// each item in the vector will be encoded and concatenated. pub fn append_vec(&mut self, vec: &Vec) - -> Result<(), EncodeError> where E: Encodable { let mut list_stream = SszStream::new(); for item in vec { item.ssz_append(&mut list_stream); } - let list_ssz = list_stream.drain(); - if list_ssz.len() <= MAX_LIST_SIZE { - self.append_encoded_val(&list_ssz); - Ok(()) - } else { - Err(EncodeError::ListTooLong) - } + self.append_encoded_val(&list_stream.drain()); } /// Consume the stream and return the underlying bytes. @@ -142,7 +135,7 @@ mod tests { fn test_encode_list() { let test_vec: Vec = vec![256; 12]; let mut stream = SszStream::new(); - stream.append_vec(&test_vec).unwrap(); + stream.append_vec(&test_vec); let ssz = stream.drain(); assert_eq!(ssz.len(), 4 + (12 * 2));