1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

dns: avoid use of arguments

Browse Source 
PR-URL: https://github.com/nodejs/node/pull/11359
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Сковорода Никита Андреевич <chalkerx@gmail.com
This commit is contained in:
James M Snell 2017-02-13 14:26:49 -08:00
parent 2ba4eeadbb
commit e36d0d3974
1 changed files with 3 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
lib/dns.js
View File

@ -56,15 +56,12 @@ function errnoException(err, syscall, hostname) {
// callback.immediately = true; // callback.immediately = true;
// } // }
function makeAsync(callback) { function makeAsync(callback) {
return function asyncCallback() { return function asyncCallback(...args) {
if (asyncCallback.immediately) { if (asyncCallback.immediately) {
// The API already returned, we can invoke the callback immediately. // The API already returned, we can invoke the callback immediately.
callback.apply(null, arguments); callback.apply(null, args);
} else { } else {
var args = new Array(arguments.length + 1); args.unshift(callback);
args[0] = callback;
for (var i = 0; i < arguments.length; ++i)
args[i + 1] = arguments[i];
process.nextTick.apply(null, args); process.nextTick.apply(null, args);
} }
}; };