Skip to content

Commit e1f00fd

Browse files
Trotttargos
authored andcommitted
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 bbc0d14 commit e1f00fd

File tree

1 file changed

+1
-19
lines changed

1 file changed

+1
-19
lines changed

benchmark/child_process/child-process-params.js

+1-19
Original file line numberDiff line numberDiff line change
@@ -82,6 +82,7 @@ function main({ n, methodName, params }) {
8282
}
8383
break;
8484
case 'execFileSync':
85+
case 'spawnSync':
8586
switch (params) {
8687
case 1:
8788
bench.start();
@@ -119,24 +120,5 @@ function main({ n, methodName, params }) {
119120
break;
120121
}
121122
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;
141123
}
142124
}

0 commit comments

Comments
 (0)