benchmark: use commas in non-csv rate output

mscdex · jasnell · commit 17eb8fce0548 · 2016-12-23T23:22:59.000-08:00
PR-URL: nodejs#10360 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: James M Snell <jasnell@gmail.com>
diff --git a/benchmark/common.js b/benchmark/common.js
@@ -185,7 +185,9 @@ function formatResult(data) {
     conf += ' ' + key + '=' + JSON.stringify(data.conf[key]);
   }
 
-  return `${data.name}${conf}: ${data.rate}`;
+  const rate = Math.floor(data.rate)
+                   .toString().replace(/(\d)(?=(\d\d\d)+(?!\d))/g, '$1,');
+  return `${data.name}${conf}: ${rate}`;
 }
 
 function sendResult(data) {
diff --git a/benchmark/run.js b/benchmark/run.js
@@ -56,7 +56,9 @@ if (format === 'csv') {
       conf = conf.replace(/"/g, '""');
       console.log(`"${data.name}", "${conf}", ${data.rate}, ${data.time}`);
     } else {
-      console.log(`${data.name} ${conf}: ${data.rate}`);
+      const rate = Math.floor(data.rate)
+                       .toString().replace(/(\d)(?=(\d\d\d)+(?!\d))/g, '$1,');
+      console.log(`${data.name} ${conf}: ${rate}`);
     }
   });
 

Original file line number	Diff line number	Diff line change
`@@ -185,7 +185,9 @@ function formatResult(data) {`
`185`	`185`	`conf += ' ' + key + '=' + JSON.stringify(data.conf[key]);`
`186`	`186`	`}`
`187`	`187`
`188`		- return `${data.name}${conf}: ${data.rate}`;
	`188`	`+ const rate = Math.floor(data.rate)`
	`189`	`+ .toString().replace(/(\d)(?=(\d\d\d)+(?!\d))/g, '$1,');`
	`190`	+ return `${data.name}${conf}: ${rate}`;
`189`	`191`	`}`
`190`	`192`
`191`	`193`	`function sendResult(data) {`