stream: avoid unecessary nextTick

If we are not going to emit 'close' then there is no reason to
schedule it.

PR-URL: nodejs#29194
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Backport-PR-URL: nodejs#29691

Author: ronag

lib/internal/streams/destroy.js

@@ -35,19 +35,25 @@ function destroy(err, cb) {
   }
 
   this._destroy(err || null, (err) => {
+    const emitClose = (this._writableState && this._writableState.emitClose) ||
+      (this._readableState && this._readableState.emitClose);
     if (!cb && err) {
       if (!this._writableState) {
-        process.nextTick(emitErrorAndCloseNT, this, err);
+        const emitNT = emitClose ? emitErrorAndCloseNT : emitErrorNT;
+        process.nextTick(emitNT, this, err);
       } else if (!this._writableState.errorEmitted) {
         this._writableState.errorEmitted = true;
-        process.nextTick(emitErrorAndCloseNT, this, err);
-      } else {
+        const emitNT = emitClose ? emitErrorAndCloseNT : emitErrorNT;
+        process.nextTick(emitNT, this, err);
+      } else if (emitClose) {
         process.nextTick(emitCloseNT, this);
       }
     } else if (cb) {
-      process.nextTick(emitCloseNT, this);
+      if (emitClose) {
+        process.nextTick(emitCloseNT, this);
+      }
       cb(err);
-    } else {
+    } else if (emitClose) {
       process.nextTick(emitCloseNT, this);
     }
   });
@@ -56,18 +62,18 @@ function destroy(err, cb) {
 }
 
 function emitErrorAndCloseNT(self, err) {
-  emitErrorNT(self, err);
-  emitCloseNT(self);
+  self.emit('error', err);
+  self.emit('close');
 }
 
 function emitCloseNT(self) {
-  if (self._writableState && !self._writableState.emitClose)
-    return;
-  if (self._readableState && !self._readableState.emitClose)
-    return;
   self.emit('close');
 }
 
+function emitErrorNT(self, err) {
+  self.emit('error', err);
+}
+
 function undestroy() {
   if (this._readableState) {
     this._readableState.destroyed = false;
@@ -87,10 +93,6 @@ function undestroy() {
   }
 }
 
-function emitErrorNT(self, err) {
-  self.emit('error', err);
-}
-
 function errorOrDestroy(stream, err) {
   // We have tests that rely on errors being emitted
   // in the same tick, so changing this is semver major.