Add EventEmitter.removeListener

nodejs · Nov 13, 2009 · bd6c08a · bd6c08a
1 parent 04f9c9f
commit bd6c08a
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 0 deletions.
diff --git a/doc/api.txt b/doc/api.txt
@@ -207,6 +207,9 @@ server.addListener("connection", function (socket) {
 });
 ----------------------------------------
 
++emitter.removeListener(event, listener)+ ::
+Remove a listener from the listener array for the specified event.
+*Caution*: changes array indices in the listener array behind the listener.
 
 +emitter.listeners(event)+ ::
 Returns an array of listeners for the specified event. This array can be

diff --git a/src/node.js b/src/node.js
@@ -201,6 +201,17 @@ process.EventEmitter.prototype.addListener = function (type, listener) {
   return this;
 };
 
+process.EventEmitter.prototype.removeListener = function (type, listener) {
+  if (listener instanceof Function) {
+    // does not use listeners(), so no side effect of creating _events[type]
+    if (!this._events || !this._events.hasOwnProperty(type)) return;
+    var list = this._events[type];
+    if (list.indexOf(listener) < 0) return;
+    list.splice(list.indexOf(listener), 1);
+  }
+  return this;
+};
+
 process.EventEmitter.prototype.listeners = function (type) {
   if (!this._events) this._events = {};
   if (!this._events.hasOwnProperty(type)) this._events[type] = [];