From ac9d9f4e9f05142f0877bcc89279369f36378a9d Mon Sep 17 00:00:00 2001 From: Michaeljohn Clement Date: Tue, 20 Apr 2010 13:07:55 -0400 Subject: [PATCH] better function test in addEventListener --- src/node.js | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/node.js b/src/node.js index 6c900880ef..3f88036004 100644 --- a/src/node.js +++ b/src/node.js @@ -110,8 +110,8 @@ var eventsModule = createInternalModule('events', function (exports) { // process.EventEmitter is defined in src/node_events.cc // process.EventEmitter.prototype.emit() is also defined there. process.EventEmitter.prototype.addListener = function (type, listener) { - if (!(listener instanceof Function)) { - throw new Error('addListener only takes instances of Function'); + if (typeof listener != 'function') { + throw new Error('addListener only takes functions'); } if (!this._events) this._events = {}; @@ -135,8 +135,8 @@ var eventsModule = createInternalModule('events', function (exports) { }; process.EventEmitter.prototype.removeListener = function (type, listener) { - if (!(listener instanceof Function)) { - throw new Error('removeListener only takes instances of Function'); + if (typeof listener != 'function') { + throw new Error('removeListener only takes functions'); } // does not use listeners(), so no side effect of creating _events[type]