No need to map twice

georust · Oct 8, 2023 · 015baad · 015baad
1 parent 88b1592
commit 015baad
Showing 1 changed file with 2 additions and 6 deletions.
diff --git a/rstar-benches/benches/benchmarks.rs b/rstar-benches/benches/benchmarks.rs
@@ -68,13 +68,9 @@ fn bulk_load_complex_geom_cached(c: &mut Criterion) {
     c.bench_function(
         "Bulk load complex geo-types geom with cached envelope",
         move |b| {
-            let polys: Vec<_> =
-                create_random_polygons(DEFAULT_BENCHMARK_TREE_SIZE, 4096, SEED_1).collect();
-            let cached: Vec<_> = polys
-                .into_iter()
-                .map(|poly| CachedEnvelope::new(poly))
+            let cached: Vec<_> = create_random_polygons(DEFAULT_BENCHMARK_TREE_SIZE, 4096, SEED_1)
+                .map(CachedEnvelope::new)
                 .collect();
-
             b.iter(|| {
                 RTree::<CachedEnvelope<_>, Params>::bulk_load_with_params(cached.clone());
             });