From dc97d372a0c9cecda5e259349c52acf1aae194be Mon Sep 17 00:00:00 2001 From: Tobin Harding Date: Mon, 18 Apr 2022 10:03:20 +1000 Subject: [PATCH] Use assert_eq for encoded data length check According to the Rust docs for `debug_assert` it should only be used if there a proved performance costs to using `assert`. Checking the length of a vector is fast, therefore its unlikely that using `assert_eq` will have any noticeable performance costs. --- src/consensus/encode.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/consensus/encode.rs b/src/consensus/encode.rs index 6be2d0caa1..a7676e6ae6 100644 --- a/src/consensus/encode.rs +++ b/src/consensus/encode.rs @@ -140,7 +140,7 @@ impl From for Error { pub fn serialize(data: &T) -> Vec { let mut encoder = Vec::new(); let len = data.consensus_encode(&mut encoder).expect("in-memory writers don't error"); - debug_assert_eq!(len, encoder.len()); + assert_eq!(len, encoder.len()); encoder }