Merge #52

52: Make sure benchmarks can be built r=cuviper a=cuviper This fixes the rand seeds, and builds benches in nightly CI. Co-authored-by: Josh Stone <[email protected]>
rust-num · May 30, 2018 · 9309c8b · 9309c8b
2 parents f87f2e7 + 40a8e91
commit 9309c8b
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 2 deletions.
diff --git a/benches/bigint.rs b/benches/bigint.rs
@@ -1,4 +1,5 @@
 #![feature(test)]
+#![cfg(feature = "rand")]
 
 extern crate test;
 extern crate num_bigint;
@@ -12,7 +13,10 @@ use num_traits::{Zero, One, FromPrimitive, Num};
 use rand::{SeedableRng, StdRng};
 
 fn get_rng() -> StdRng {
-    let seed: &[_] = &[1, 2, 3, 4];
+    let mut seed = [0; 32];
+    for i in 1..32 {
+        seed[usize::from(i)] = i;
+    }
     SeedableRng::from_seed(seed)
 }
 

diff --git a/benches/gcd.rs b/benches/gcd.rs
@@ -1,4 +1,5 @@
 #![feature(test)]
+#![cfg(feature = "rand")]
 
 extern crate test;
 extern crate num_bigint;
@@ -13,7 +14,10 @@ use num_traits::Zero;
 use rand::{SeedableRng, StdRng};
 
 fn get_rng() -> StdRng {
-    let seed: &[_] = &[1, 2, 3, 4];
+    let mut seed = [0; 32];
+    for i in 1..32 {
+        seed[usize::from(i)] = i;
+    }
     SeedableRng::from_seed(seed)
 }
 

diff --git a/ci/test_full.sh b/ci/test_full.sh
@@ -29,3 +29,8 @@ done
 # test all supported features together
 cargo build --features="std $FEATURES"
 cargo test --features="std $FEATURES"
+
+# make sure benchmarks can be built
+if [[ "$TRAVIS_RUST_VERSION" == "nightly" ]]; then
+  cargo bench --all-features --no-run
+fi