Merge pull request #12 from hendriknielaender/calc-avg

fix: use duration array length for average calculation
hendriknielaender · Nov 3, 2023 · d358be3 · d358be3
2 parents e83dc4c + 2163b60
commit d358be3
Showing 1 changed file with 5 additions and 4 deletions.
diff --git a/zbench.zig b/zbench.zig
@@ -168,15 +168,16 @@ pub const Benchmark = struct {
 
     // Calculate the average duration
     pub fn calculateAverage(self: Benchmark) u64 {
-        // prevent a div by zero: check number of ops
-        const ops = self.totalOperations;
-        if (ops == 0) return 0;
+        // prevent division by zero
+        const len = self.durations.items.len;
+        if (len == 0) return 0;
 
         var sum: u64 = 0;
         for (self.durations.items) |duration| {
             sum += duration;
         }
-        const avg = sum / ops; // TODO : make sure n ops == self.durations.items.len
+
+        const avg = sum / len;
 
         return avg;
     }