Skip to content
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.

Commit 9da3f21

Browse files
tniessenjasnell
authored andcommittedJan 25, 2021
benchmark: make output RFC 4180 compliant
PR-URL: nodejs#37038 Reviewed-By: Darshan Sen <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Colin Ihrig <[email protected]>
1 parent cd1e73a commit 9da3f21

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed
 

‎benchmark/compare.js

+3-3
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,7 @@ for (const filename of benchmarks) {
5656
// queue.length = binary.length * runs * benchmarks.length
5757

5858
// Print csv header
59-
console.log('"binary", "filename", "configuration", "rate", "time"');
59+
console.log('"binary","filename","configuration","rate","time"');
6060

6161
const kStartOfQueue = 0;
6262

@@ -85,8 +85,8 @@ if (showProgress) {
8585
// Escape quotes (") for correct csv formatting
8686
conf = conf.replace(/"/g, '""');
8787

88-
console.log(`"${job.binary}", "${job.filename}", "${conf}", ` +
89-
`${data.rate}, ${data.time}`);
88+
console.log(`"${job.binary}","${job.filename}","${conf}",` +
89+
`${data.rate},${data.time}`);
9090
if (showProgress) {
9191
// One item in the subqueue has been completed.
9292
progress.completeConfig(data);

0 commit comments

Comments
 (0)