EventEmitter docs should indicate that the emitter must be instantiated via util.inherits() and EventEmitter.call(this)

[yunong]

The current EventEmitter docs: http://nodejs.org/api/events.html do not indicate how one should instantiate an instance of the emitter.

The blessed procedure is:

tj.fontaine@joyent.com
11:53 you should util.inherits and EventEmitter.call(this)

[dougwilson]

Right now it is only semi-documented on https://github.com/joyent/node/wiki/Api-changes-between-v0.8-and-v0.10

[bnoordhuis]

Semi-blessed, perhaps. MyClass.prototype = new EventEmitter or MyClass.prototype = Object.create(EventEmitter.prototype) are supposed to work as well but that's broken in v0.10, see #7157. Fortunately, it's fixed in master, see #7405.

[vkurchatkin]

@bnoordhuis MyClass.prototype = Object.create(EventEmitter.prototype) should be ok

@yunong using inherits is not mandatory, any correct javascript inheritance pattern will work

[brendanashworth]

This was fixed in nodejs/node#2168 - and because I don't think the doc change will be backported, I'll close this issue for now.

[targos]

@brendanashworth even if an issue is still relevant or a fix has to be backported to 0.12 or 0.10, it would be done in nodejs/node.

[brendanashworth]

@targos hmm.. is there any reason to leave any of these issues open then? :)

[targos]

The only reason I see is for you are doing right now: answering to them, closing and redirecting to nodejs/node if applicable.