From a08fcc02f8549635d6c79c11545777ce07ec185a Mon Sep 17 00:00:00 2001 From: Brian White Date: Wed, 8 Feb 2017 03:27:35 -0500 Subject: benchmark: fix timer display in progress output This commit fixes an issue where the minutes would not display properly on the benchmark timer once an hour had elapsed. PR-URL: https://github.com/nodejs/node/pull/11235 Reviewed-By: Joyee Cheung Reviewed-By: Andreas Madsen Reviewed-By: James M Snell --- benchmark/_benchmark_progress.js | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'benchmark/_benchmark_progress.js') diff --git a/benchmark/_benchmark_progress.js b/benchmark/_benchmark_progress.js index 2a2a458c5c..4b42248f24 100644 --- a/benchmark/_benchmark_progress.js +++ b/benchmark/_benchmark_progress.js @@ -15,9 +15,9 @@ function fraction(numerator, denominator) { function getTime(diff) { const time = Math.ceil(diff[0] + diff[1] / 1e9); - const seconds = pad(time % 60, 2, '0'); - const minutes = pad(Math.floor(time / 60) % (60 * 60), 2, '0'); - const hours = pad(Math.floor(time / (60 * 60)), 2, '0'); + const hours = pad(Math.floor(time / 3600), 2, '0'); + const minutes = pad(Math.floor((time % 3600) / 60), 2, '0'); + const seconds = pad((time % 3600) % 60, 2, '0'); return `${hours}:${minutes}:${seconds}`; } -- cgit v1.2.3