From d3e23e18fc31e80837ec71d9082c682d3629ce9a Mon Sep 17 00:00:00 2001 From: Matthias Radestock Date: Thu, 7 Dec 2017 20:31:11 +0000 Subject: [PATCH] do not start/stop benchmark timer for no-ops --- app/benchmark_internal_test.go | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/app/benchmark_internal_test.go b/app/benchmark_internal_test.go index afb603568..a46b02953 100644 --- a/app/benchmark_internal_test.go +++ b/app/benchmark_internal_test.go @@ -42,8 +42,6 @@ func readReportFiles(path string) ([]report.Report, error) { func BenchmarkReportUnmarshal(b *testing.B) { b.ResetTimer() for i := 0; i < b.N; i++ { - b.StopTimer() - b.StartTimer() if _, err := readReportFiles(*benchReportPath); err != nil { b.Fatal(err) } @@ -58,9 +56,8 @@ func BenchmarkReportMerge(b *testing.B) { merger := NewSmartMerger() b.ResetTimer() + for i := 0; i < b.N; i++ { - b.StopTimer() - b.StartTimer() merger.Merge(reports) } }