From 256be7f3a6bf998d91a1d96e8c90e9bf8aa9c788 Mon Sep 17 00:00:00 2001 From: Jonas Jenwald Date: Fri, 27 Mar 2026 15:28:45 +0100 Subject: [PATCH] Remove manual loops in the `StatTimer.prototype.toString` method We can use Array methods instead, which is a tiny bit shorter. --- src/display/display_utils.js | 14 +++++--------- 1 file changed, 5 insertions(+), 9 deletions(-) diff --git a/src/display/display_utils.js b/src/display/display_utils.js index a05b47182..cfe1c6ad4 100644 --- a/src/display/display_utils.js +++ b/src/display/display_utils.js @@ -443,15 +443,11 @@ class StatTimer { toString() { // Find the longest name for padding purposes. - const outBuf = []; - let longest = 0; - for (const { name } of this.times) { - longest = Math.max(name.length, longest); - } - for (const { name, start, end } of this.times) { - outBuf.push(`${name.padEnd(longest)} ${end - start}ms\n`); - } - return outBuf.join(""); + const longest = Math.max(...this.times.map(t => t.name.length)); + + return this.times + .map(t => `${t.name.padEnd(longest)} ${t.end - t.start}ms\n`) + .join(""); } }