credo and format fixes

lib/benchee/formatters/render.ex

@@ -106,9 +106,7 @@ defmodule Benchee.Formatters.HTML.Render do
   end
 
   defp format_mode(modes = [_ | _], unit) do
-    modes
-    |> Enum.map(fn mode -> format_property(mode, unit) end)
-    |> Enum.join(", ")
+    Enum.map_join(modes, ", ", fn mode -> format_property(mode, unit) end)
   end
 
   defp format_mode(value, unit) do

samples/fast_memory.exs

@@ -1,10 +1,11 @@
 list = Enum.to_list(1..10_000)
-map_fun = fn(i) -> [i, i * i] end
+map_fun = fn i -> [i, i * i] end
 
-Benchee.run(%{
-  "flat_map"    => fn -> Enum.flat_map(list, map_fun) end,
-  "map.flatten" => fn -> list |> Enum.map(map_fun) |> List.flatten end
-},
+Benchee.run(
+  %{
+    "flat_map" => fn -> Enum.flat_map(list, map_fun) end,
+    "map.flatten" => fn -> list |> Enum.map(map_fun) |> List.flatten() end
+  },
   formatters: [
     {Benchee.Formatters.HTML, file: "samples_output/memory.html"},
     Benchee.Formatters.Console

samples/fast_memory_changing.exs

@@ -1,14 +1,15 @@
 # random by design so that memory actually changes
 # this has too wide of a range and too small of a sample size, don't do this at home
-Benchee.run(%{
-  "Enum.to_list"    => fn range -> Enum.to_list(range) end,
-  "Enum.into" => fn range -> Enum.into(range, []) end
-},
+Benchee.run(
+  %{
+    "Enum.to_list" => fn range -> Enum.to_list(range) end,
+    "Enum.into" => fn range -> Enum.into(range, []) end
+  },
   formatters: [
     {Benchee.Formatters.HTML, file: "samples_output/memory_changing.html"},
     {Benchee.Formatters.Console, extended_statistics: true}
   ],
-  before_each: fn _ -> (0..:rand.uniform(1_000) + 1000) end,
+  before_each: fn _ -> 0..(:rand.uniform(1_000) + 1000) end,
   warmup: 0,
   time: 0.1,
   memory_time: 0.1

samples/many_jobs.exs

@@ -1,21 +1,24 @@
-list_10k  = 1..10_000    |> Enum.to_list |> Enum.shuffle
-list_30k  = 1..30_000    |> Enum.to_list |> Enum.shuffle
-list_50k  = 1..50_000    |> Enum.to_list |> Enum.shuffle
-list_70k  = 1..70_000    |> Enum.to_list |> Enum.shuffle
-list_90k  = 1..90_000    |> Enum.to_list |> Enum.shuffle
-list_100k = 1..100_000   |> Enum.to_list |> Enum.shuffle
+list_10k = 1..10_000 |> Enum.to_list() |> Enum.shuffle()
+list_30k = 1..30_000 |> Enum.to_list() |> Enum.shuffle()
+list_50k = 1..50_000 |> Enum.to_list() |> Enum.shuffle()
+list_70k = 1..70_000 |> Enum.to_list() |> Enum.shuffle()
+list_90k = 1..90_000 |> Enum.to_list() |> Enum.shuffle()
+list_100k = 1..100_000 |> Enum.to_list() |> Enum.shuffle()
 
-Benchee.run %{
-  "10k"  => fn -> Enum.sort(list_10k) end,
-  "30k"  => fn -> Enum.sort(list_30k) end,
-  "50k"  => fn -> Enum.sort(list_50k) end,
-  "70k"  => fn -> Enum.sort(list_70k) end,
-  "90k"  => fn -> Enum.sort(list_90k) end,
-  "100k" => fn -> Enum.sort(list_100k) end,
-}, time:   2,
-   warmup: 0,
-   unit_scaling: :largest,
-   formatters: [
-     Benchee.Formatters.Console,
-     {Benchee.Formatters.HTML, file: "samples_output/many.html"}
-   ]
+Benchee.run(
+  %{
+    "10k" => fn -> Enum.sort(list_10k) end,
+    "30k" => fn -> Enum.sort(list_30k) end,
+    "50k" => fn -> Enum.sort(list_50k) end,
+    "70k" => fn -> Enum.sort(list_70k) end,
+    "90k" => fn -> Enum.sort(list_90k) end,
+    "100k" => fn -> Enum.sort(list_100k) end
+  },
+  time: 2,
+  warmup: 0,
+  unit_scaling: :largest,
+  formatters: [
+    Benchee.Formatters.Console,
+    {Benchee.Formatters.HTML, file: "samples_output/many.html"}
+  ]
+)

samples/multiple_inputs.exs

@@ -1,9 +1,10 @@
-map_fun = fn(i) -> [i, i * i] end
+map_fun = fn i -> [i, i * i] end
 
-Benchee.run(%{
-  "flat_map"    => fn(list) -> Enum.flat_map(list, map_fun) end,
-  "map.flatten" => fn(list) -> list |> Enum.map(map_fun) |> List.flatten end
-},
+Benchee.run(
+  %{
+    "flat_map" => fn list -> Enum.flat_map(list, map_fun) end,
+    "map.flatten" => fn list -> list |> Enum.map(map_fun) |> List.flatten() end
+  },
   formatters: [
     {Benchee.Formatters.HTML, file: "samples_output/multiputs.html"},
     Benchee.Formatters.Console
@@ -12,6 +13,6 @@ Benchee.run(%{
   warmup: 3,
   inputs: %{
     "Smaller List" => Enum.to_list(1..1_000),
-    "Bigger List"  => Enum.to_list(1..100_000),
+    "Bigger List" => Enum.to_list(1..100_000)
   }
 )