From 7866508482c39d1289a901a41ec6978fb8345e03 Mon Sep 17 00:00:00 2001 From: Joyee Cheung Date: Mon, 11 Mar 2019 10:19:47 +0300 Subject: [PATCH] timers: reduce usage of public util Refs: https://github.com/nodejs/node/issues/26546 PR-URL: https://github.com/nodejs/node/pull/26583 Reviewed-By: Anna Henningsen --- lib/timers.js | 19 +++++++++++-------- 1 file changed, 11 insertions(+), 8 deletions(-) diff --git a/lib/timers.js b/lib/timers.js index cb7eb5e5afd..30bc646698b 100644 --- a/lib/timers.js +++ b/lib/timers.js @@ -39,8 +39,11 @@ const { initAsyncResource, validateTimerDuration } = require('internal/timers'); -const internalUtil = require('internal/util'); -const util = require('util'); +const { + promisify: { custom: customPromisify }, + deprecate +} = require('internal/util'); +const { inspect } = require('internal/util/inspect'); const { ERR_INVALID_CALLBACK } = require('internal/errors').codes; let debuglog; @@ -249,8 +252,8 @@ function TimersList(expiry, msecs) { } // Make sure the linked list only shows the minimal necessary information. -TimersList.prototype[util.inspect.custom] = function(_, options) { - return util.inspect(this, { +TimersList.prototype[inspect.custom] = function(_, options) { + return inspect(this, { ...options, // Only inspect one level. depth: 0, @@ -457,7 +460,7 @@ function setTimeout(callback, after, arg1, arg2, arg3) { return timeout; } -setTimeout[internalUtil.promisify.custom] = function(after, value) { +setTimeout[customPromisify] = function(after, value) { const args = value !== undefined ? [value] : value; return new Promise((resolve) => { active(new Timeout(resolve, after, args, false)); @@ -719,7 +722,7 @@ function setImmediate(callback, arg1, arg2, arg3) { return new Immediate(callback, args); } -setImmediate[internalUtil.promisify.custom] = function(value) { +setImmediate[customPromisify] = function(value) { return new Promise((resolve) => new Immediate(resolve, [value])); }; @@ -752,11 +755,11 @@ module.exports = { clearImmediate, setInterval, clearInterval, - unenroll: util.deprecate( + unenroll: deprecate( unenroll, 'timers.unenroll() is deprecated. Please use clearTimeout instead.', 'DEP0096'), - enroll: util.deprecate( + enroll: deprecate( enroll, 'timers.enroll() is deprecated. Please use setTimeout instead.', 'DEP0095')