Commit 4318e70
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 456fd75 commit 4318e70
1 file changed
Lines changed: 3 additions & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -56,7 +56,7 @@ for (const filename of benchmarks) { | |||
| 56 | 56 | // queue.length = binary.length * runs * benchmarks.length | |
| 57 | 57 | ||
| 58 | 58 | // Print csv header | |
| 59 | - console.log('"binary", "filename", "configuration", "rate", "time"'); | ||
| 59 | + console.log('"binary","filename","configuration","rate","time"'); | ||
| 60 | 60 | ||
| 61 | 61 | const kStartOfQueue = 0; | |
| 62 | 62 | ||
@@ -85,8 +85,8 @@ if (showProgress) { | |||
| 85 | 85 | // Escape quotes (") for correct csv formatting | |
| 86 | 86 | conf = conf.replace(/"/g, '""'); | |
| 87 | 87 | ||
| 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}`); | ||
| 90 | 90 | if (showProgress) { | |
| 91 | 91 | // One item in the subqueue has been completed. | |
| 92 | 92 | progress.completeConfig(data); | |
0 commit comments