stream: removes unnecessary params

Removes the state param in the onFinished function since it's never used within it. PR-URL: #32936 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Gerhard Stöbich <[email protected]>
nodejs · Apr 28, 2020 · c21f1f0 · c21f1f0
1 parent 4b6aa07
commit c21f1f0
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/lib/_stream_writable.js b/lib/_stream_writable.js
@@ -582,7 +582,7 @@ Writable.prototype.end = function(chunk, encoding, cb) {
     if (err || state.finished)
       process.nextTick(cb, err);
     else
-      onFinished(this, state, cb);
+      onFinished(this, cb);
   }
 
   return this;
@@ -664,7 +664,7 @@ function finish(stream, state) {
 }
 
 // TODO(ronag): Avoid using events to implement internal logic.
-function onFinished(stream, state, cb) {
+function onFinished(stream, cb) {
   function onerror(err) {
     stream.removeListener('finish', onfinish);
     stream.removeListener('error', onerror);