Switch condition order to check for null before calling isNaN().
Also remove two unnecessary calls to isNaN() that are already
covered by calls to isFinite(). This commit targets v0.10, as
opposed to #7891, which targets master (suggested by
@bnoordhuis). Closes #7840.
Signed-off-by: Fedor Indutny <fedor@indutny.com>
if (state.objectMode)
return n === 0 ? 0 : 1;
- if (isNaN(n) || n === null) {
+ if (n === null || isNaN(n)) {
// only flow one buffer at a time
if (state.flowing && state.buffer.length)
return state.buffer[0].length;
if (value === undefined) {
return '' + value;
}
- if (typeof value === 'number' && (isNaN(value) || !isFinite(value))) {
+ if (typeof value === 'number' && !isFinite(value)) {
return value.toString();
}
if (typeof value === 'function' || value instanceof RegExp) {
Socket.prototype.setTimeout = function(msecs, callback) {
- if (msecs > 0 && !isNaN(msecs) && isFinite(msecs)) {
+ if (msecs > 0 && isFinite(msecs)) {
timers.enroll(this, msecs);
timers._unrefActive(this);
if (callback) {