From 4318e708b8b9c45cb5ae8302e52a8f5e74e90c2b Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Tobias=20Nie=C3=9Fen?= Date: Sat, 23 Jan 2021 15:58:24 +0100 Subject: [PATCH] benchmark: make output RFC 4180 compliant PR-URL: https://github.com/nodejs/node/pull/37038 Reviewed-By: Darshan Sen Reviewed-By: James M Snell Reviewed-By: Rich Trott Reviewed-By: Luigi Pinca Reviewed-By: Antoine du Hamel Reviewed-By: Colin Ihrig --- benchmark/compare.js | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/benchmark/compare.js b/benchmark/compare.js index 5c9cd03be3fdee..169948e006d660 100644 --- a/benchmark/compare.js +++ b/benchmark/compare.js @@ -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; @@ -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);