tools: use built-in padStart instead of padString

tniessen · gibfahn · commit 2fb6515db5f1 · 2017-12-19T20:45:29.000Z
PR-URL: #17120 Reviewed-By: Timothy Gu <timothygu99@gmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Anatoli Papirovski <apapirovski@mac.com> Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
diff --git a/tools/lint-js.js b/tools/lint-js.js
@@ -215,12 +215,12 @@ if (cluster.isMaster) {
 
     // Calculate and format the data for displaying
     const elapsed = process.hrtime(startTime)[0];
-    const mins = padString(Math.floor(elapsed / 60), 2, '0');
-    const secs = padString(elapsed % 60, 2, '0');
-    const passed = padString(successes, 6, ' ');
-    const failed = padString(failures, 6, ' ');
+    const mins = `${Math.floor(elapsed / 60)}`.padStart(2, '0');
+    const secs = `${elapsed % 60}`.padStart(2, '0');
+    const passed = `${successes}`.padStart(6);
+    const failed = `${failures}`.padStart(6);
     var pct = Math.ceil(((totalPaths - paths.length) / totalPaths) * 100);
-    pct = padString(pct, 3, ' ');
+    pct = `${pct}`.padStart(3);
 
     var line = `[${mins}:${secs}|%${pct}|+${passed}|-${failed}]: ${curPath}`;
 
@@ -233,13 +233,6 @@ if (cluster.isMaster) {
 
     outFn(line);
   }
-
-  function padString(str, len, chr) {
-    str = `${str}`;
-    if (str.length >= len)
-      return str;
-    return chr.repeat(len - str.length) + str;
-  }
 } else {
   // Worker
 
