Use zip + map instead of zip_with for old elixir compatibility

bencheeorg · Dec 14, 2023 · c43880d · c43880d
1 parent d01d26d
commit c43880d
Showing 1 changed file with 19 additions and 21 deletions.
diff --git a/lib/benchee/statistics.ex b/lib/benchee/statistics.ex
@@ -197,27 +197,25 @@ defmodule Benchee.Statistics do
 
   defp update_scenarios_with_statistics(scenarios, scenario_statistics) do
     # we can zip them as they retained order
-    Enum.zip_with(
-      scenarios,
-      scenario_statistics,
-      fn scenario, {run_time_stats, memory_stats, reductions_stats} ->
-        %Scenario{
-          scenario
-          | run_time_data: %CollectionData{
-              scenario.run_time_data
-              | statistics: run_time_stats
-            },
-            memory_usage_data: %CollectionData{
-              scenario.memory_usage_data
-              | statistics: memory_stats
-            },
-            reductions_data: %CollectionData{
-              scenario.reductions_data
-              | statistics: reductions_stats
-            }
-        }
-      end
-    )
+    scenarios
+    |> Enum.zip(scenario_statistics)
+    |> Enum.map(fn {scenario, {run_time_stats, memory_stats, reductions_stats}} ->
+      %Scenario{
+        scenario
+        | run_time_data: %CollectionData{
+            scenario.run_time_data
+            | statistics: run_time_stats
+          },
+          memory_usage_data: %CollectionData{
+            scenario.memory_usage_data
+            | statistics: memory_stats
+          },
+          reductions_data: %CollectionData{
+            scenario.reductions_data
+            | statistics: reductions_stats
+          }
+      }
+    end)
   end
 
   defp calculate_scenario_statistics({run_time_data, memory_data, reductions_data}, percentiles) do