Possible issue with .prototype = new EventEmitter

[AlexeyKupershtokh]

I've seen you do

zmqstream.Socket.super_ = EventEmitter
zmqstream.Socket.prototype.__proto__ = new EventEmitter()

in the lib/zmqstream.js

There were a recent bug with a similar situation: nodejs/node-v0.x-archive#4971
So you may want to check if it's applicable to zmq-stream as well.

[Schoonology]

We should be okay because we do call super_ during construction. That said, I haven't been able to test it yet, so I'm leaving this issue open.

Thanks for the find!

[AlexeyKupershtokh]

Yep, it's reproduced in node 0.10.3:

var zmqstream = require('../lib/zmqstream')
var a = new zmqstream.Socket({ type: zmqstream.Type.ROUTER });
var b = new zmqstream.Socket({ type: zmqstream.Type.ROUTER });
a.on('x', function(){ console.log('X'); });
b.emit('x');

outputs "X"