process: remove protection for SyncWriteStream destroy in stdio

#26691 introduced an if to protect against SyncWriteStream not using the default .destroy() mechanism. This change removes that as SyncWriteStream now use standard .destroy(). See: #26691 PR-URL: #26902 Reviewed-By: Ben Noordhuis <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Anna Henningsen <[email protected]>
nodejs · Apr 4, 2019 · 04a4242 · 04a4242
1 parent 6e6aad2
commit 04a4242
Showing 1 changed file with 1 addition and 7 deletions.
diff --git a/lib/internal/process/stdio.js b/lib/internal/process/stdio.js
@@ -3,13 +3,7 @@
 exports.getMainThreadStdio = getMainThreadStdio;
 
 function dummyDestroy(err, cb) {
-  // SyncWriteStream does not use the stream
-  // destroy mechanism for some legacy reason.
-  // TODO(mcollina): remove when
-  // https://github.com/nodejs/node/pull/26690 lands.
-  if (typeof cb === 'function') {
-    cb(err);
-  }
+  cb(err);
 
   // We need to emit 'close' anyway so that the closing
   // of the stream is observable. We just make sure we