From 60fc56795284f22d8e12d743f4dc66aeb3bd7ea2 Mon Sep 17 00:00:00 2001
From: Jackson Tian <shyvo1987@gmail.com>
Date: Sat, 4 Feb 2017 00:23:10 +0800
Subject: [PATCH] child_process: move anonymous class to top level

Move the anonymous class out of setupChannel to clarify code.

PR-URL: https://github.com/nodejs/node/pull/11147
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
---
 lib/internal/child_process.js | 37 ++++++++++++++++++-----------------
 1 file changed, 19 insertions(+), 18 deletions(-)

diff --git a/lib/internal/child_process.js b/lib/internal/child_process.js
index 3017cd5889a037..b3023f6a46d9b7 100644
--- a/lib/internal/child_process.js
+++ b/lib/internal/child_process.js
@@ -407,6 +407,24 @@ ChildProcess.prototype.unref = function() {
   if (this._handle) this._handle.unref();
 };
 
+class Control extends EventEmitter {
+  constructor(channel) {
+    super();
+    this.channel = channel;
+    this.refs = 0;
+  }
+  ref() {
+    if (++this.refs === 1) {
+      this.channel.ref();
+    }
+  }
+  unref() {
+    if (--this.refs === 0) {
+      this.channel.unref();
+      this.emit('unref');
+    }
+  }
+}
 
 function setupChannel(target, channel) {
   target.channel = channel;
@@ -421,24 +439,7 @@ function setupChannel(target, channel) {
   target._handleQueue = null;
   target._pendingHandle = null;
 
-  const control = new class extends EventEmitter {
-    constructor() {
-      super();
-      this.channel = channel;
-      this.refs = 0;
-    }
-    ref() {
-      if (++this.refs === 1) {
-        this.channel.ref();
-      }
-    }
-    unref() {
-      if (--this.refs === 0) {
-        this.channel.unref();
-        this.emit('unref');
-      }
-    }
-  }();
+  const control = new Control(channel);
 
   var decoder = new StringDecoder('utf8');
   var jsonBuffer = '';