http: remove unneeded cb check from setTimeout()

aks- · rvagg · commit a7f28a098ef4 · 2015-11-06T11:06:04.000+11:00
- This check is already covered in EventEmitter#addListener() Refs: #3618 PR-URL: #3631 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com> Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
diff --git a/lib/_http_outgoing.js b/lib/_http_outgoing.js
@@ -91,8 +91,6 @@ exports.OutgoingMessage = OutgoingMessage;
 OutgoingMessage.prototype.setTimeout = function(msecs, callback) {
 
   if (callback) {
-    if (typeof callback !== 'function')
-      throw new TypeError('callback must be a function');
     this.on('timeout', callback);
   }
 

Original file line number	Diff line number	Diff line change
`@@ -91,8 +91,6 @@ exports.OutgoingMessage = OutgoingMessage;`
`91`	`91`	`OutgoingMessage.prototype.setTimeout = function(msecs, callback) {`
`92`	`92`
`93`	`93`	`if (callback) {`
`94`		`- if (typeof callback !== 'function')`
`95`		`- throw new TypeError('callback must be a function');`
`96`	`94`	`this.on('timeout', callback);`
`97`	`95`	`}`
`98`	`96`