Skip to content

Commit

Permalink
timers: rename validateTimerDuration to getTimerDuration
Browse files Browse the repository at this point in the history
The function did not only validate the timer but it caused side
effects like a warning and potentially returned a different value
than the input value. Thus the name `validate` did not seem to be
appropriate.

PR-URL: #26809
Reviewed-By: Matteo Collina <[email protected]>
Reviewed-By: James M Snell <[email protected]>
  • Loading branch information
BridgeAR committed Mar 27, 2019
1 parent 6c913fb commit 9e8c9be
Show file tree
Hide file tree
Showing 4 changed files with 9 additions and 9 deletions.
6 changes: 3 additions & 3 deletions lib/_http_client.js
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ const {
ERR_INVALID_PROTOCOL,
ERR_UNESCAPED_CHARACTERS
} = require('internal/errors').codes;
const { validateTimerDuration } = require('internal/timers');
const { getTimerDuration } = require('internal/timers');
const is_reused_symbol = require('internal/freelist').symbols.is_reused_symbol;
const {
DTRACE_HTTP_CLIENT_REQUEST,
Expand Down Expand Up @@ -146,7 +146,7 @@ function ClientRequest(input, options, cb) {
this.socketPath = options.socketPath;

if (options.timeout !== undefined)
this.timeout = validateTimerDuration(options.timeout, 'timeout');
this.timeout = getTimerDuration(options.timeout, 'timeout');

var method = options.method;
var methodIsString = (typeof method === 'string');
Expand Down Expand Up @@ -743,7 +743,7 @@ ClientRequest.prototype.setTimeout = function setTimeout(msecs, callback) {
}

listenSocketTimeout(this);
msecs = validateTimerDuration(msecs, 'msecs');
msecs = getTimerDuration(msecs, 'msecs');
if (callback) this.once('timeout', callback);

if (this.socket) {
Expand Down
4 changes: 2 additions & 2 deletions lib/internal/stream_base_commons.js
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ const { owner_symbol } = require('internal/async_hooks').symbols;
const {
kTimeout,
setUnrefTimeout,
validateTimerDuration
getTimerDuration
} = require('internal/timers');

const kMaybeDestroy = Symbol('kMaybeDestroy');
Expand Down Expand Up @@ -205,7 +205,7 @@ function setStreamTimeout(msecs, callback) {
this.timeout = msecs;

// Type checking identical to timers.enroll()
msecs = validateTimerDuration(msecs, 'msecs');
msecs = getTimerDuration(msecs, 'msecs');

// Attempt to clear an existing timer in both cases -
// even if it will be rescheduled we don't want to leak an existing timer.
Expand Down
4 changes: 2 additions & 2 deletions lib/internal/timers.js
Original file line number Diff line number Diff line change
Expand Up @@ -359,7 +359,7 @@ function setUnrefTimeout(callback, after) {
}

// Type checking used by timers.enroll() and Socket#setTimeout()
function validateTimerDuration(msecs, name) {
function getTimerDuration(msecs, name) {
validateNumber(msecs, name);
if (msecs < 0 || !isFinite(msecs)) {
throw new ERR_OUT_OF_RANGE(name, 'a non-negative finite number', msecs);
Expand Down Expand Up @@ -586,7 +586,7 @@ module.exports = {
kRefed,
initAsyncResource,
setUnrefTimeout,
validateTimerDuration,
getTimerDuration,
immediateQueue,
getTimerCallbacks,
immediateInfoFields: {
Expand Down
4 changes: 2 additions & 2 deletions lib/timers.js
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ const {
},
kRefed,
initAsyncResource,
validateTimerDuration,
getTimerDuration,
timerListMap,
timerListQueue,
immediateQueue,
Expand Down Expand Up @@ -102,7 +102,7 @@ function unenroll(item) {
// This function does not start the timer, see `active()`.
// Using existing objects as timers slightly reduces object overhead.
function enroll(item, msecs) {
msecs = validateTimerDuration(msecs, 'msecs');
msecs = getTimerDuration(msecs, 'msecs');

// If this item was already in a list somewhere
// then we should unenroll it from that
Expand Down

0 comments on commit 9e8c9be

Please sign in to comment.