Skip to content

Commit 29d0b43

Browse files
apapirovskidanbev
authored andcommitted
benchmark: fix http headers benchmark
PR-URL: #27021 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Gus Caplan <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: James M Snell <[email protected]>
1 parent 47f5cc1 commit 29d0b43

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

benchmark/http/headers.js

+3-1
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,9 @@ function main({ len, n }) {
1414
'Transfer-Encoding': 'chunked',
1515
};
1616

17-
const Is = [ ...Array(n / len).keys() ];
17+
// TODO(BridgeAR): Change this benchmark to use grouped arguments when
18+
// implemented. https://github.com/nodejs/node/issues/26425
19+
const Is = [ ...Array(Math.max(n / len, 1)).keys() ];
1820
const Js = [ ...Array(len).keys() ];
1921
for (const i of Is) {
2022
headers[`foo${i}`] = Js.map(() => `some header value ${i}`);

0 commit comments

Comments
 (0)