From fbb8cf62f5f6429651cd5c7c6add141a685fb7bd Mon Sep 17 00:00:00 2001 From: Brian Smith Date: Fri, 24 Nov 2023 18:01:37 -0800 Subject: [PATCH] bigint: Fold `Modulus` constructors together. Now we have exactly one way to construct a `Modulus` so we can simplify the constructors. --- src/arithmetic/bigint/modulus.rs | 9 +-------- 1 file changed, 1 insertion(+), 8 deletions(-) diff --git a/src/arithmetic/bigint/modulus.rs b/src/arithmetic/bigint/modulus.rs index b0750639a..dc7a9f880 100644 --- a/src/arithmetic/bigint/modulus.rs +++ b/src/arithmetic/bigint/modulus.rs @@ -95,14 +95,7 @@ impl OwnedModulus { input: untrusted::Input, cpu_features: cpu::Features, ) -> Result { - let limbs = BoxedLimbs::positive_minimal_width_from_be_bytes(input)?; - Self::from_boxed_limbs(limbs, cpu_features) - } - - fn from_boxed_limbs( - n: BoxedLimbs, - cpu_features: cpu::Features, - ) -> Result { + let n = BoxedLimbs::positive_minimal_width_from_be_bytes(input)?; if n.len() > MODULUS_MAX_LIMBS { return Err(error::KeyRejected::too_large()); }