Skip to content

Commit

Permalink
Don't copy LinuxPerf Stats
Browse files Browse the repository at this point in the history
It's not clear to me how deep the copying should be, but I think this should be fine for now and it matches the semantics of `copy` better.
  • Loading branch information
Zentrik committed Jan 8, 2024
1 parent bbfb733 commit 336b0fd
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/trials.jl
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ function Base.copy(t::Trial)
copy(t.gctimes),
t.memory,
t.allocs,
isnothing(t.linux_perf_stats) ? nothing : copy(t.linux_perf_stats),
t.linux_perf_stats,
)
end

Expand Down Expand Up @@ -137,7 +137,7 @@ function Base.copy(t::TrialEstimate)
t.gctime,
t.memory,
t.allocs,
isnothing(t.linux_perf_stats) ? nothing : copy(t.linux_perf_stats),
t.linux_perf_stats,
)
end

Expand Down

0 comments on commit 336b0fd

Please sign in to comment.