child_process: simplify send() result handling

PR-URL: #13459 Reviewed-By: Refael Ackermann <refack@gmail.com> Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
nodejs · Jun 9, 2017 · dd83d11 · dd83d11
1 parent 8f91aa5
commit dd83d11
Showing 1 changed file with 7 additions and 8 deletions.
diff --git a/lib/internal/child_process.js b/lib/internal/child_process.js
@@ -645,16 +645,15 @@ function setupChannel(target, channel) {
           obj.postSend(handle, options, target);
       }
 
-      req.oncomplete = function() {
-        if (this.async === true)
+      if (req.async) {
+        req.oncomplete = function() {
           control.unref();
-        if (typeof callback === 'function')
-          callback(null);
-      };
-      if (req.async === true) {
+          if (typeof callback === 'function')
+            callback(null);
+        };
         control.ref();
-      } else {
-        process.nextTick(function() { req.oncomplete(); });
+      } else if (typeof callback === 'function') {
+        process.nextTick(callback, null);
       }
     } else {
       // Cleanup handle on error