Skip to content

lib: consistent this in callbacks #14645

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
Trott wants to merge 5 commits into from
Closed

lib: consistent this in callbacks #14645

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
b3166e3
test: refactor test-fs-stat
Trott Aug 6, 2017
9d6365a
fs: invoke callbacks with undefined context
Trott Aug 6, 2017
23918c4
test: check `this` value for `nextTick()`
Trott Aug 6, 2017
aa07e87
process: make `this` value consistent
Trott Aug 6, 2017
64e1f00
benchmark: cover more nextTick() code
Trott Aug 6, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
benchmark: cover more nextTick() code 
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.
  • Loading branch information
@Trott
Trott committed Aug 9, 2017
commit 64e1f005cd9287894ce16a4c644d158c272b7714
9 changes: 8 additions & 1 deletion benchmark/process/next-tick-breadth-args.js
View file
Open in desktop
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
View file
Open in desktop
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