stream: removes unnecessary params

JesuHrz · BethGriggs · commit c21f1f03c5f2 · 2020-04-28T16:28:32.000+01:00
Removes the state param in the onFinished function since it's never used within it. PR-URL: #32936 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Gerhard Stöbich <deb2001-github@yahoo.de>
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);

-Original file line number
+Diff line change
 if(err||state.finished)
 process.nextTick(cb,err);
 else
 -onFinished(this,state,cb);
 +onFinished(this,cb);
+}
 returnthis;
+}
 // TODO(ronag): Avoid using events to implement internal logic.
 -functiononFinished(stream,state,cb){
 +functiononFinished(stream,cb){
 functiononerror(err){
 stream.removeListener('finish',onfinish);
 stream.removeListener('error',onerror);