Merge pull request stacks-network#6480 from Jiloc/fix/remove-duplicate-CostOverflowingMath

Fix:remove duplicated `CostOverflowingMath` trait

Author: Jiloc

clarity/src/vm/costs/mod.rs

@@ -18,7 +18,7 @@ use std::collections::HashMap;
 use std::{cmp, fmt};
 
 pub use clarity_serialization::errors::CostErrors;
-pub use clarity_serialization::execution_cost::ExecutionCost;
+pub use clarity_serialization::execution_cost::{CostOverflowingMath, ExecutionCost};
 use costs_1::Costs1;
 use costs_2::Costs2;
 use costs_2_testnet::Costs2Testnet;
@@ -1253,28 +1253,6 @@ impl CostTracker for &mut LimitedCostTracker {
     }
 }
 
-pub trait CostOverflowingMath<T> {
-    fn cost_overflow_mul(self, other: T) -> Result<T>;
-    fn cost_overflow_add(self, other: T) -> Result<T>;
-    fn cost_overflow_sub(self, other: T) -> Result<T>;
-    fn cost_overflow_div(self, other: T) -> Result<T>;
-}
-
-impl CostOverflowingMath<u64> for u64 {
-    fn cost_overflow_mul(self, other: u64) -> Result<u64> {
-        self.checked_mul(other).ok_or(CostErrors::CostOverflow)
-    }
-    fn cost_overflow_add(self, other: u64) -> Result<u64> {
-        self.checked_add(other).ok_or(CostErrors::CostOverflow)
-    }
-    fn cost_overflow_sub(self, other: u64) -> Result<u64> {
-        self.checked_sub(other).ok_or(CostErrors::CostOverflow)
-    }
-    fn cost_overflow_div(self, other: u64) -> Result<u64> {
-        self.checked_div(other).ok_or(CostErrors::CostOverflow)
-    }
-}
-
 // ONLY WORKS IF INPUT IS u64
 fn int_log2(input: u64) -> Option<u64> {
     63_u32.checked_sub(input.leading_zeros()).map(|floor_log| {