util.inherits cannot be used with ES6 classes

[zen-crab]

I'm experimenting with ES6 classes and noticed I was unable to use util.inherits to inherit EventEmitter into the prototype of my class.

'use strict';
var util = require('util');
var EventEmitter = require('events').EventEmitter;

class x {
    constructor() {
        EventEmitter.call(this);
    }
    emitEvent() {
        this.emit('event emitted!');
    }
}

util.inherits(x, EventEmitter); // Fails

The error is TypeError: Cannot assign to read only property 'prototype' of class x { constructor() { EventEmitter.call(this); } emitEvent() { this.emit('event emitted!'); } }

I was able to get inheritance to work using class x extends EventEmitter, but using extends means my class can't be a subclass of anything besides EventEmitter unless I make EventEmitter the root class.

Thoughts on this?

[bnoordhuis]

I think util.inherits() would need to start using Object.setPrototypeOf() instead of assigning to the .prototype property directly? Not sure how it should interact with the .constructor property it tries to set.

[targos]

@bnoordhuis I just tried your suggestion and it looks good. I'll add a test and PR it if you don't mind.

[targos]

#3455

[targos]

@sbmelvin to be clear, #3455 just allows to use util.inherits on ES6 classes (and cleans a little bit the implementation). The result will be similar to what you get with extends.
It seems from your post that you would like to do multiple inheritance, such as

class X {}
class Y extends X {}
util.inherits(Y, EventEmitter);

This is not possible. util.inherits overwrites the prototype chain of Y, removing the inheritance on X.

[zen-crab]

That makes sense. Thanks.