From 20e721cefd801ac8471c2e76acafcf04e960667e Mon Sep 17 00:00:00 2001 From: Jacob Jenkins Date: Fri, 2 Oct 2015 16:38:52 -0400 Subject: [PATCH] Add optional string parameter to removeAllListeners As socket essentially extends EventEmitter (save emit), and EventEmitter allows passing an event string to removeAllListeners, the typings should be updated to reflect that. I've updated removeAllListeners to take an option event string parameter. Reference [Node 4.1.1](https://nodejs.org/api/events.html#events_emitter_removealllisteners_event) & [Node 0.12.7](https://nodejs.org/docs/latest-v0.12.x/api/events.html#events_emitter_removealllisteners_event) --- socket.io/socket.io.d.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/socket.io/socket.io.d.ts b/socket.io/socket.io.d.ts index a5b13044fe..a34a78b8c4 100644 --- a/socket.io/socket.io.d.ts +++ b/socket.io/socket.io.d.ts @@ -687,7 +687,7 @@ declare module SocketIO { * Removes all event listeners on this object * @return This Socket */ - removeAllListeners(): Socket; + removeAllListeners( event?: string ): Socket; /** * Sets the maximum number of listeners this instance can have