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

timers: remove unused variable

Browse Source 
A recent commit removed the usage of the second argument of
tryOnTimeout but left the definition in place. Remove it.

PR-URL: https://github.com/nodejs/node/pull/18579
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
This commit is contained in:
Anatoli Papirovski 2018-02-05 09:47:01 -05:00
parent b1c6ecb2c6
commit 1b05d7bc86
No known key found for this signature in database
GPG Key ID: 614E2E1ABEB4B2C0
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
lib/timers.js
View File

@ -260,7 +260,7 @@ TimerWrap.prototype[kOnTimeout] = function listOnTimeout(now) {
continue; continue;
} }
tryOnTimeout(timer, list); tryOnTimeout(timer);
} }
// If `L.peek(list)` returned nothing, the list was either empty or we have // If `L.peek(list)` returned nothing, the list was either empty or we have
@ -289,7 +289,7 @@ TimerWrap.prototype[kOnTimeout] = function listOnTimeout(now) {
// An optimization so that the try/finally only de-optimizes (since at least v8 // An optimization so that the try/finally only de-optimizes (since at least v8
// 4.7) what is in this smaller function. // 4.7) what is in this smaller function.
function tryOnTimeout(timer, list) { function tryOnTimeout(timer) {
timer._called = true; timer._called = true;
const timerAsyncId = (typeof timer[async_id_symbol] === 'number') ? const timerAsyncId = (typeof timer[async_id_symbol] === 'number') ?
timer[async_id_symbol] : null; timer[async_id_symbol] : null;