Avoid clippy ignores with saturating math

paulhauner · Feb 4, 2025 · 2d39282 · 2d39282
1 parent 01e1c17
commit 2d39282
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/consensus/types/src/checkpoint.rs b/consensus/types/src/checkpoint.rs
@@ -42,9 +42,10 @@ impl Encode for Checkpoint {
         self.root.ssz_append(buf);
     }
 
-    #[allow(clippy::arithmetic_side_effects)]
     fn ssz_bytes_len(&self) -> usize {
-        self.epoch.ssz_bytes_len() + self.root.ssz_bytes_len()
+        self.epoch
+            .ssz_bytes_len()
+            .saturating_add(self.root.ssz_bytes_len())
     }
 }
 
@@ -54,9 +55,8 @@ impl Decode for Checkpoint {
         true
     }
 
-    #[allow(clippy::arithmetic_side_effects)]
     fn ssz_fixed_len() -> usize {
-        <Epoch as Decode>::ssz_fixed_len() + <Hash256 as Decode>::ssz_fixed_len()
+        <Epoch as Decode>::ssz_fixed_len().saturating_add(<Hash256 as Decode>::ssz_fixed_len())
     }
 
     fn from_ssz_bytes(bytes: &[u8]) -> Result<Self, ssz::DecodeError> {