Use Object.defineProperty() when overriding properties from a frozen prototype.

Author: felthy

.gitignore

@@ -1,2 +1,3 @@
 node_modules
 npm-debug.log
+/.idea

lib/cat.js

@@ -63,9 +63,9 @@ function State(emitter, streams) {
   // We `bind()` the event listener callback methods, so that they
   // get an appropriate `this` when they're called during event
   // emission.
-  this.onEnd = this.onEnd.bind(this);
-  this.onData = this.onData.bind(this);
-  this.onError = this.onError.bind(this);
+  Object.defineProperty(this, 'onEnd', { value: this.onEnd.bind(this), enumerable: true });
+  Object.defineProperty(this, 'onData', { value: this.onData.bind(this), enumerable: true });
+  Object.defineProperty(this, 'onError', { value: this.onError.bind(this), enumerable: true });
 
   if (!Array.isArray(streams)) {
     throw new Error("Invalid streams array.");

lib/dropper.js

@@ -76,9 +76,9 @@ function State(emitter, source, options) {
   // We `bind()` the event listener callback methods, so that they
   // get an appropriate `this` when they're called during event
   // emission.
-  this.onCloseOrEnd = this.onCloseOrEnd.bind(this);
-  this.onData = this.onData.bind(this);
-  this.onError = this.onError.bind(this);
+  Object.defineProperty(this, 'onCloseOrEnd', { value: this.onCloseOrEnd.bind(this), enumerable: true });
+  Object.defineProperty(this, 'onData', { value: this.onData.bind(this), enumerable: true });
+  Object.defineProperty(this, 'onError', { value: this.onError.bind(this), enumerable: true });
 
   source.on(consts.CLOSE, this.onCloseOrEnd);
   source.on(consts.DATA, this.onData);

lib/sink.js

@@ -78,10 +78,10 @@ function State(emitter, source) {
   // We `bind()` the event listener callback methods, so that they
   // get an appropriate `this` when they're called during event
   // emission.
-  this.onClose = this.onClose.bind(this);
-  this.onData = this.onData.bind(this);
-  this.onEnd = this.onEnd.bind(this);
-  this.onError = this.onError.bind(this);
+  Object.defineProperty(this, 'onClose', { value: this.onClose.bind(this), enumerable: true });
+  Object.defineProperty(this, 'onData', { value: this.onData.bind(this), enumerable: true });
+  Object.defineProperty(this, 'onEnd', { value: this.onEnd.bind(this), enumerable: true });
+  Object.defineProperty(this, 'onError', { value: this.onError.bind(this), enumerable: true });
 
   source.on(consts.CLOSE, this.onClose);
   source.on(consts.DATA, this.onData);

lib/slicer.js

@@ -190,10 +190,10 @@ function State(source) {
   // We `bind()` the event listener callback methods, so that they
   // get an appropriate `this` when they're called during event
   // emission.
-  this.onClose = this.onClose.bind(this);
-  this.onData = this.onData.bind(this);
-  this.onEnd = this.onEnd.bind(this);
-  this.onError = this.onError.bind(this);
+  Object.defineProperty(this, 'onClose', { value: this.onClose.bind(this), enumerable: true });
+  Object.defineProperty(this, 'onData', { value: this.onData.bind(this), enumerable: true });
+  Object.defineProperty(this, 'onEnd', { value: this.onEnd.bind(this), enumerable: true });
+  Object.defineProperty(this, 'onError', { value: this.onError.bind(this), enumerable: true });
 
   source.on(consts.CLOSE, this.onClose);
   source.on(consts.DATA, this.onData);

lib/valve.js

@@ -70,10 +70,10 @@ function State(emitter, source) {
   // We `bind()` the event listener callback methods, so that they
   // get an appropriate `this` when they're called during event
   // emission.
-  this.onClose = this.onClose.bind(this);
-  this.onData = this.onData.bind(this);
-  this.onEnd = this.onEnd.bind(this);
-  this.onError = this.onError.bind(this);
+  Object.defineProperty(this, 'onClose', { value: this.onClose.bind(this), enumerable: true });
+  Object.defineProperty(this, 'onData', { value: this.onData.bind(this), enumerable: true });
+  Object.defineProperty(this, 'onEnd', { value: this.onEnd.bind(this), enumerable: true });
+  Object.defineProperty(this, 'onError', { value: this.onError.bind(this), enumerable: true });
 
   source.on(consts.CLOSE, this.onClose);
   source.on(consts.DATA, this.onData);