Remove an unnecessary allocation from the trie router benchmark (#447)

hummingbird-project · May 13, 2024 · 05a4590 · 05a4590
1 parent 5a6a097
commit 05a4590
Showing 1 changed file with 9 additions and 3 deletions.
diff --git a/Benchmarks/Benchmarks/Router/TrieRouterBenchmarks.swift b/Benchmarks/Benchmarks/Router/TrieRouterBenchmarks.swift
@@ -28,7 +28,9 @@ func trieRouterBenchmarks() {
         benchmark.startMeasurement()
 
         for _ in benchmark.scaledIterations {
-            blackHole(testValues.map { trie.resolve($0) })
+            for testValue in testValues {
+                blackHole(trie.resolve(testValue))
+            }
         }
     } setup: {
         let trieBuilder = RouterPathTrieBuilder<String>()
@@ -53,7 +55,9 @@ func trieRouterBenchmarks() {
         benchmark.startMeasurement()
 
         for _ in benchmark.scaledIterations {
-            blackHole(testValues.map { trie2.resolve($0) })
+            for testValue in testValues {
+                blackHole(trie2.resolve(testValue))
+            }
         }
     } setup: {
         let trieBuilder = RouterPathTrieBuilder<String>()
@@ -73,7 +77,9 @@ func trieRouterBenchmarks() {
         benchmark.startMeasurement()
 
         for _ in benchmark.scaledIterations {
-            blackHole(testValues.map { trie3.resolve($0) })
+            for testValue in testValues {
+                blackHole(trie3.resolve(testValue))
+            }
         }
     } setup: {
         let trieBuilder = RouterPathTrieBuilder<String>()