Clean up and format

Author: Zentrik

src/parameters.jl

@@ -26,11 +26,8 @@ function perf_available()
 
     try
         opts = LinuxPerf.parse_pstats_options([])
-        groups = BenchmarkTools.LinuxPerf.set_default_spaces(
-            opts.events,
-            opts.spaces,
-        )
-        bench = make_bench_threaded(groups, threads = opts.threads)
+        groups = BenchmarkTools.LinuxPerf.set_default_spaces(opts.events, opts.spaces)
+        bench = make_bench_threaded(groups; threads=opts.threads)
         return true
     catch
         return false
@@ -139,9 +136,9 @@ function Base.copy(p::Parameters)
         p.memory_tolerance,
         p.experimental_enable_linux_perf,
         (
-            events = copy(p.linux_perf_options.events),
-            spaces = copy(p.linux_perf_options.spaces),
-            threads = copy(p.linux_perf_options.threads)
+            events=copy(p.linux_perf_options.events),
+            spaces=copy(p.linux_perf_options.spaces),
+            threads=copy(p.linux_perf_options.threads),
         ),
     )
 end

src/serialization.jl

@@ -40,14 +40,17 @@ Base.convert(::Type{LinuxPerf.Stats}, d::Dict{String}) = LinuxPerf.Stats(d["thre
 function Base.convert(::Type{LinuxPerf.ThreadStats}, d::Dict{String})
     return LinuxPerf.ThreadStats(d["pid"], d["groups"])
 end
-JSON.lower(counter::LinuxPerf.Counter) = [counter.event.category, counter.event.event, counter.value, counter.enabled, counter.running]
+function JSON.lower(counter::LinuxPerf.Counter)
+    return [
+        counter.event.category,
+        counter.event.event,
+        counter.value,
+        counter.enabled,
+        counter.running,
+    ]
+end
 function Base.convert(::Type{LinuxPerf.Counter}, v::Vector)
-    return LinuxPerf.Counter(
-        LinuxPerf.EventType(v[1], v[2]),
-        v[3],
-        v[4],
-        v[5],
-    )
+    return LinuxPerf.Counter(LinuxPerf.EventType(v[1], v[2]), v[3], v[4], v[5])
 end
 
 # a minimal 'eval' function, mirroring KeyTypes, but being slightly more lenient
@@ -191,7 +194,7 @@ function load(io::IO, args...)
             ),
         )
     end
-    parsed = JSON.parse(io; dicttype=Dict)
+    parsed = JSON.parse(io)
     if !isa(parsed, Vector) ||
         length(parsed) != 2 ||
         !isa(parsed[1], Dict) ||

src/trials.jl

@@ -134,7 +134,9 @@ mutable struct TrialEstimate
 end
 
 function TrialEstimate(trial::Trial, t, gct)
-    return TrialEstimate(params(trial), t, gct, memory(trial), allocs(trial), trial.linux_perf_stats)
+    return TrialEstimate(
+        params(trial), t, gct, memory(trial), allocs(trial), trial.linux_perf_stats
+    )
 end
 
 # Should we compare linux_perf_stats here?