Sakurai/pdf.js
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Slightly re-factor the String handling in StatTimer

Browse Source 
This uses template strings in a couple of spots, and a buffer in the `toString` method.
This commit is contained in:
Jonas Jenwald 2019-10-23 13:45:31 +02:00
parent 8e5d3836d6
commit 6f7f8257bc
1 changed files with 5 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
src/display/display_utils.js
View File

@ -387,14 +387,14 @@ class StatTimer {
time(name) { time(name) {
if (name in this.started) { if (name in this.started) {
warn('Timer is already running for ' + name); warn(`Timer is already running for ${name}`);
} }
this.started[name] = Date.now(); this.started[name] = Date.now();
} }
timeEnd(name) { timeEnd(name) {
if (!(name in this.started)) { if (!(name in this.started)) {
warn('Timer has not been started for ' + name); warn(`Timer has not been started for ${name}`);
} }
this.times.push({ this.times.push({
'name': name, 'name': name,
@ -407,7 +407,7 @@ class StatTimer {
toString() { toString() {
// Find the longest name for padding purposes. // Find the longest name for padding purposes.
let out = '', longest = 0; let outBuf = [], longest = 0;
for (const time of this.times) { for (const time of this.times) {
const name = time.name; const name = time.name;
if (name.length > longest) { if (name.length > longest) {
@ -416,9 +416,9 @@ class StatTimer {
} }
for (const time of this.times) { for (const time of this.times) {
const duration = time.end - time.start; const duration = time.end - time.start;
out += `${time.name.padEnd(longest)} ${duration}ms\n`; outBuf.push(`${time.name.padEnd(longest)} ${duration}ms\n`);
} }
return out; return outBuf.join('');
} }
} }