Skip to content

Commit

Permalink
benchmark: cover more nextTick() code
Browse files Browse the repository at this point in the history
The benchmarks for `process.nextTick()` do not cover the `default` case
in the internal code's `switch` statement where the callback receives
more than 3 arguments. Modify two of the benchmarks to include this
condition.

PR-URL: #14645
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
  • Loading branch information
Trott committed Aug 10, 2017
1 parent a253704 commit 97c4394
Show file tree
Hide file tree
Showing 2 changed files with 30 additions and 4 deletions.
9 changes: 8 additions & 1 deletion benchmark/process/next-tick-breadth-args.js
Original file line number Diff line number Diff line change
Expand Up @@ -24,10 +24,17 @@ function main(conf) {
if (n === N)
bench.end(n / 1e6);
}
function cb4(arg1, arg2, arg3, arg4) {
n++;
if (n === N)
bench.end(n / 1e6);
}

bench.start();
for (var i = 0; i < N; i++) {
if (i % 3 === 0)
if (i % 4 === 0)
process.nextTick(cb4, 3.14, 1024, true, false);
else if (i % 3 === 0)
process.nextTick(cb3, 512, true, null);
else if (i % 2 === 0)
process.nextTick(cb2, false, 5.1);
Expand Down
25 changes: 22 additions & 3 deletions benchmark/process/next-tick-depth-args.js
Original file line number Diff line number Diff line change
Expand Up @@ -10,9 +10,24 @@ process.maxTickDepth = Infinity;
function main(conf) {
var n = +conf.millions * 1e6;

function cb4(arg1, arg2, arg3, arg4) {
if (--n) {
if (n % 4 === 0)
process.nextTick(cb4, 3.14, 1024, true, false);
else if (n % 3 === 0)
process.nextTick(cb3, 512, true, null);
else if (n % 2 === 0)
process.nextTick(cb2, false, 5.1);
else
process.nextTick(cb1, 0);
} else
bench.end(+conf.millions);
}
function cb3(arg1, arg2, arg3) {
if (--n) {
if (n % 3 === 0)
if (n % 4 === 0)
process.nextTick(cb4, 3.14, 1024, true, false);
else if (n % 3 === 0)
process.nextTick(cb3, 512, true, null);
else if (n % 2 === 0)
process.nextTick(cb2, false, 5.1);
Expand All @@ -23,7 +38,9 @@ function main(conf) {
}
function cb2(arg1, arg2) {
if (--n) {
if (n % 3 === 0)
if (n % 4 === 0)
process.nextTick(cb4, 3.14, 1024, true, false);
else if (n % 3 === 0)
process.nextTick(cb3, 512, true, null);
else if (n % 2 === 0)
process.nextTick(cb2, false, 5.1);
Expand All @@ -34,7 +51,9 @@ function main(conf) {
}
function cb1(arg1) {
if (--n) {
if (n % 3 === 0)
if (n % 4 === 0)
process.nextTick(cb4, 3.14, 1024, true, false);
else if (n % 3 === 0)
process.nextTick(cb3, 512, true, null);
else if (n % 2 === 0)
process.nextTick(cb2, false, 5.1);
Expand Down

0 comments on commit 97c4394

Please sign in to comment.