search

grncdr / merge-stream

Merge multiple streams into one interleaved stream
MIT License
214 stars 16 forks source link

EventEmitter leak error #3

Closed yocontra closed 10 years ago

yocontra commented 10 years ago

Too many streams will give you this. If you look at event-stream's merge it removes the listener on the outgoing stream to fix this.

https://github.com/dominictarr/event-stream/blob/master/index.js#L35

grncdr commented 10 years ago

Fixed in 0.1.2

grncdr commented 10 years ago

Btw, thanks for reporting this and #2 to me. TBH I don't even remember what I used this module for when I wrote it, but it might as well work :wink: