Commit 681c4af
benchmark: reduce code duplication
Merge duplicate case code in child-process-params.js.
PR-URL: #36568
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>1 parent dcbcd9e commit 681c4af
1 file changed
Lines changed: 1 addition & 19 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -82,6 +82,7 @@ function main({ n, methodName, params }) { | |||
| 82 | 82 | } | |
| 83 | 83 | break; | |
| 84 | 84 | case 'execFileSync': | |
| 85 | + case 'spawnSync': | ||
| 85 | 86 | switch (params) { | |
| 86 | 87 | case 1: | |
| 87 | 88 | bench.start(); | |
@@ -119,24 +120,5 @@ function main({ n, methodName, params }) { | |||
| 119 | 120 | break; | |
| 120 | 121 | } | |
| 121 | 122 | break; | |
| 122 | - case 'spawnSync': | ||
| 123 | - switch (params) { | ||
| 124 | - case 1: | ||
| 125 | - bench.start(); | ||
| 126 | - for (let i = 0; i < n; i++) method(command); | ||
| 127 | - bench.end(n); | ||
| 128 | - break; | ||
| 129 | - case 2: | ||
| 130 | - bench.start(); | ||
| 131 | - for (let i = 0; i < n; i++) method(command, args); | ||
| 132 | - bench.end(n); | ||
| 133 | - break; | ||
| 134 | - case 3: | ||
| 135 | - bench.start(); | ||
| 136 | - for (let i = 0; i < n; i++) method(command, args, options); | ||
| 137 | - bench.end(n); | ||
| 138 | - break; | ||
| 139 | - } | ||
| 140 | - break; | ||
| 141 | 123 | } | |
| 142 | 124 | } | |
0 commit comments