timers: set several methods EOL · nodejs/node@5d7091f · GitHub
Skip to content

Commit 5d7091f

Browse files
authored
timers: set several methods EOL
PR-URL: #56966 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com> Reviewed-By: Chemi Atlow <chemi@atlow.co.il> Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com>
1 parent 3ea97d5 commit 5d7091f

10 files changed

Lines changed: 22 additions & 262 deletions

doc/api/deprecations.md

Lines changed: 20 additions & 8 deletions

lib/timers.js

Lines changed: 0 additions & 35 deletions
Original file line numberDiff line numberDiff line change
@@ -46,18 +46,14 @@ const {
4646
},
4747
kRefed,
4848
kHasPrimitive,
49-
getTimerDuration,
5049
timerListMap,
5150
timerListQueue,
5251
immediateQueue,
53-
active,
54-
unrefActive,
5552
insert,
5653
knownTimersById,
5754
} = require('internal/timers');
5855
const {
5956
promisify: { custom: customPromisify },
60-
deprecate,
6157
} = require('internal/util');
6258
let debug = require('internal/util/debuglog').debuglog('timer', (fn) => {
6359
debug = fn;
@@ -111,20 +107,6 @@ function unenroll(item) {
111107
item._idleTimeout = -1;
112108
}
113109

114-
// Make a regular object able to act as a timer by setting some properties.
115-
// This function does not start the timer, see `active()`.
116-
// Using existing objects as timers slightly reduces object overhead.
117-
function enroll(item, msecs) {
118-
msecs = getTimerDuration(msecs, 'msecs');
119-
120-
// If this item was already in a list somewhere
121-
// then we should unenroll it from that
122-
if (item._idleNext) unenroll(item);
123-
124-
L.init(item);
125-
item._idleTimeout = msecs;
126-
}
127-
128110

129111
/**
130112
* Schedules the execution of a one-time `callback`
@@ -351,23 +333,6 @@ module.exports = timers = {
351333
clearImmediate,
352334
setInterval,
353335
clearInterval,
354-
_unrefActive: deprecate(
355-
unrefActive,
356-
'timers._unrefActive() is deprecated.' +
357-
' Please use timeout.refresh() instead.',
358-
'DEP0127'),
359-
active: deprecate(
360-
active,
361-
'timers.active() is deprecated. Please use timeout.refresh() instead.',
362-
'DEP0126'),
363-
unenroll: deprecate(
364-
unenroll,
365-
'timers.unenroll() is deprecated. Please use clearTimeout instead.',
366-
'DEP0096'),
367-
enroll: deprecate(
368-
enroll,
369-
'timers.enroll() is deprecated. Please use setTimeout instead.',
370-
'DEP0095'),
371336
};
372337

373338
ObjectDefineProperties(timers, {

test/parallel/test-timers-active.js

Lines changed: 0 additions & 34 deletions
This file was deleted.

test/parallel/test-timers-enroll-invalid-msecs.js

Lines changed: 0 additions & 38 deletions
This file was deleted.

test/parallel/test-timers-enroll-second-time.js

Lines changed: 0 additions & 16 deletions
This file was deleted.

test/parallel/test-timers-max-duration-warning.js

Lines changed: 1 addition & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@
22

33
const common = require('../common');
44
const assert = require('assert');
5-
const timers = require('timers');
65

76
const OVERFLOW = Math.pow(2, 31); // TIMEOUT_MAX is 2^31-1
87

@@ -19,7 +18,7 @@ process.on('warning', common.mustCall((warning) => {
1918
assert.strictEqual(lines[0], `${OVERFLOW} does not fit into a 32-bit signed` +
2019
' integer.');
2120
assert.strictEqual(lines.length, 2);
22-
}, 6));
21+
}, 2));
2322

2423

2524
{
@@ -31,12 +30,3 @@ process.on('warning', common.mustCall((warning) => {
3130
const interval = setInterval(timerNotCanceled, OVERFLOW);
3231
clearInterval(interval);
3332
}
34-
35-
{
36-
const timer = {
37-
_onTimeout: timerNotCanceled
38-
};
39-
timers.enroll(timer, OVERFLOW);
40-
timers.active(timer);
41-
timers.unenroll(timer);
42-
}

test/parallel/test-timers-unenroll-unref-interval.js

Lines changed: 1 addition & 2 deletions

test/parallel/test-timers-unref-active.js

Lines changed: 0 additions & 47 deletions
This file was deleted.

test/parallel/test-timers-unref-remove-other-unref-timers-only-one-fires.js

Lines changed: 0 additions & 41 deletions
This file was deleted.

0 commit comments

Comments
 (0)