Skip to content

Commit ec22756

Browse files
tniessentargos
authored andcommitted
benchmark: make output RFC 4180 compliant
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>
1 parent a7b6464 commit ec22756

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)