Skip to content

Commit

Permalink
events: remove unreachable code
Browse files Browse the repository at this point in the history
Commit 8d386ed stopped the
Event Emitter implementation from storing arrays containing a
single listener. This change left a section of code in
removeListener() as unreachable. This commit removes the
unreachable code.

Refs: nodejs#12043
PR-URL: nodejs#12501
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
  • Loading branch information
cjihrig committed Apr 21, 2017
1 parent 1159a71 commit 58066d1
Showing 1 changed file with 5 additions and 14 deletions.
19 changes: 5 additions & 14 deletions lib/events.js
Original file line number Diff line number Diff line change
Expand Up @@ -374,22 +374,13 @@ EventEmitter.prototype.removeListener =
if (position < 0)
return this;

if (list.length === 1) {
if (--this._eventsCount === 0) {
this._events = Object.create(null);
return this;
} else {
delete events[type];
}
} else if (position === 0) {
if (position === 0)
list.shift();
if (list.length === 1)
events[type] = list[0];
} else {
else
spliceOne(list, position);
if (list.length === 1)
events[type] = list[0];
}

if (list.length === 1)
events[type] = list[0];

if (events.removeListener)
this.emit('removeListener', type, originalListener || listener);
Expand Down

0 comments on commit 58066d1

Please sign in to comment.