Skip to content

Commit

Permalink
benchmark: make output RFC 4180 compliant
Browse files Browse the repository at this point in the history
PR-URL: #37038
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
  • Loading branch information
tniessen authored and targos committed May 1, 2021
1 parent 456fd75 commit 4318e70
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions benchmark/compare.js
Expand Up @@ -56,7 +56,7 @@ for (const filename of benchmarks) {
// queue.length = binary.length * runs * benchmarks.length

// Print csv header
console.log('"binary", "filename", "configuration", "rate", "time"');
console.log('"binary","filename","configuration","rate","time"');

const kStartOfQueue = 0;

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

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

0 comments on commit 4318e70

Please sign in to comment.