From 4d090855c6199cb1308ed88afbdfbf691533c758 Mon Sep 17 00:00:00 2001
From: James M Snell <jasnell@gmail.com>
Date: Sun, 26 Feb 2017 17:54:40 -0800
Subject: [PATCH] lib: avoid using forEach in LazyTransform

PR-URL: https://github.com/nodejs/node/pull/11582
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
---
 lib/internal/streams/lazy_transform.js | 66 ++++++++++++++++----------
 1 file changed, 41 insertions(+), 25 deletions(-)

diff --git a/lib/internal/streams/lazy_transform.js b/lib/internal/streams/lazy_transform.js
index 3607d98534..06cc87143d 100644
--- a/lib/internal/streams/lazy_transform.js
+++ b/lib/internal/streams/lazy_transform.js
@@ -14,31 +14,47 @@ function LazyTransform(options) {
 }
 util.inherits(LazyTransform, stream.Transform);
 
-[
-  '_readableState',
-  '_writableState',
-  '_transformState'
-].forEach(function(prop, i, props) {
-  Object.defineProperty(LazyTransform.prototype, prop, {
-    get: function() {
-      stream.Transform.call(this, this._options);
-      this._writableState.decodeStrings = false;
-
-      if (!this._options || !this._options.defaultEncoding) {
-        this._writableState.defaultEncoding = crypto.DEFAULT_ENCODING;
-      }
-
-      return this[prop];
-    },
-    set: function(val) {
-      Object.defineProperty(this, prop, {
-        value: val,
-        enumerable: true,
-        configurable: true,
-        writable: true
-      });
-    },
+function makeGetter(name) {
+  return function() {
+    stream.Transform.call(this, this._options);
+    this._writableState.decodeStrings = false;
+
+    if (!this._options || !this._options.defaultEncoding) {
+      this._writableState.defaultEncoding = crypto.DEFAULT_ENCODING;
+    }
+
+    return this[name];
+  };
+}
+
+function makeSetter(name) {
+  return function(val) {
+    Object.defineProperty(this, name, {
+      value: val,
+      enumerable: true,
+      configurable: true,
+      writable: true
+    });
+  };
+}
+
+Object.defineProperties(LazyTransform.prototype, {
+  _readableState: {
+    get: makeGetter('_readableState'),
+    set: makeSetter('_readableState'),
+    configurable: true,
+    enumerable: true
+  },
+  _writableState: {
+    get: makeGetter('_writableState'),
+    set: makeSetter('_writableState'),
+    configurable: true,
+    enumerable: true
+  },
+  _transformState: {
+    get: makeGetter('_transformState'),
+    set: makeSetter('_transformState'),
     configurable: true,
     enumerable: true
-  });
+  }
 });