Commit 4d78eb8
benchmark: improve compare output
The current output uses JSON.stringify to escape the config values.
This switches to util.inspect to have a better readable output.
PR-URL: #18597
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Anatoli Papirovski <apapirovski@mac.com>
Reviewed-By: Andreas Madsen <amwebdk@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>1 parent ba46103 commit 4d78eb8
1 file changed
Lines changed: 3 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -1,6 +1,7 @@ | |||
| 1 | 1 | 'use strict'; | |
| 2 | 2 | ||
| 3 | - const fork = require('child_process').fork; | ||
| 3 | + const { fork } = require('child_process'); | ||
| 4 | + const { inspect } = require('util'); | ||
| 4 | 5 | const path = require('path'); | |
| 5 | 6 | const CLI = require('./_cli.js'); | |
| 6 | 7 | const BenchmarkProgress = require('./_benchmark_progress.js'); | |
@@ -76,7 +77,7 @@ if (showProgress) { | |||
| 76 | 77 | // Construct configuration string, " A=a, B=b, ..." | |
| 77 | 78 | let conf = ''; | |
| 78 | 79 | for (const key of Object.keys(data.conf)) { | |
| 79 | - conf += ` ${key}=${JSON.stringify(data.conf[key])}`; | ||
| 80 | + conf += ` ${key}=${inspect(data.conf[key])}`; | ||
| 80 | 81 | } | |
| 81 | 82 | conf = conf.slice(1); | |
| 82 | 83 | // Escape quotes (") for correct csv formatting | |
0 commit comments