remove unnecessary call to isNaN()

teppeis · teppeis · commit e2cf7f3d8460 · 2014-12-21T19:15:44.000+09:00
from nodejs/node-v0.x-archive@b87ca79#diff-1
diff --git a/assert.js b/assert.js
@@ -86,7 +86,7 @@ function replacer(key, value) {
   if (util.isUndefined(value)) {
     return '' + value;
   }
-  if (util.isNumber(value) && (isNaN(value) || !isFinite(value))) {
+  if (util.isNumber(value) && !isFinite(value)) {
     return value.toString();
   }
   if (util.isFunction(value) || util.isRegExp(value)) {

Original file line number	Diff line number	Diff line change
`@@ -86,7 +86,7 @@ function replacer(key, value) {`
`86`	`86`	`if (util.isUndefined(value)) {`
`87`	`87`	`return '' + value;`
`88`	`88`	`}`
`89`		`- if (util.isNumber(value) && (isNaN(value) \|\| !isFinite(value))) {`
	`89`	`+ if (util.isNumber(value) && !isFinite(value)) {`
`90`	`90`	`return value.toString();`
`91`	`91`	`}`
`92`	`92`	`if (util.isFunction(value) \|\| util.isRegExp(value)) {`