Merge #82

82: overwrite Zero/One default impls for `set_zero` and `set_one` r=cuviper a=lcnr

title

Co-authored-by: lcnr/Bastian Kauschke <bastian_kauschke@hotmail.de>
Co-authored-by: Josh Stone <cuviper@gmail.com>

Cargo.toml

@@ -38,7 +38,7 @@ version = "0.1.39"
 default-features = false
 
 [dependencies.num-traits]
-version = "0.2.4"
+version = "0.2.7"
 default-features = false
 
 [dependencies.rand]

src/bigint.rs

@@ -756,6 +756,12 @@ impl Zero for BigInt {
         BigInt::from_biguint(NoSign, Zero::zero())
     }
 
+    #[inline]
+    fn set_zero(&mut self) {
+        self.data.set_zero();
+        self.sign = NoSign;
+    }
+
     #[inline]
     fn is_zero(&self) -> bool {
         self.sign == NoSign
@@ -768,6 +774,12 @@ impl One for BigInt {
         BigInt::from_biguint(Plus, One::one())
     }
 
+    #[inline]
+    fn set_one(&mut self) {
+        self.data.set_one();
+        self.sign = Plus;
+    }
+
     #[inline]
     fn is_one(&self) -> bool {
         self.sign == Plus && self.data.is_one()

src/biguint.rs

@@ -417,6 +417,11 @@ impl Zero for BigUint {
         BigUint::new(Vec::new())
     }
 
+    #[inline]
+    fn set_zero(&mut self) {
+        self.data.clear();
+    }
+
     #[inline]
     fn is_zero(&self) -> bool {
         self.data.is_empty()
@@ -429,6 +434,12 @@ impl One for BigUint {
         BigUint::new(vec![1])
     }
 
+    #[inline]
+    fn set_one(&mut self) {
+        self.data.clear();
+        self.data.push(1);
+    }
+
     #[inline]
     fn is_one(&self) -> bool {
         self.data[..] == [1]