Skip to content

Commit c21f1f0

Browse files
JesuHrzBethGriggs
authored andcommitted
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]>
1 parent 4b6aa07 commit c21f1f0

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

lib/_stream_writable.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -582,7 +582,7 @@ Writable.prototype.end = function(chunk, encoding, cb) {
582582
if (err || state.finished)
583583
process.nextTick(cb, err);
584584
else
585-
onFinished(this, state, cb);
585+
onFinished(this, cb);
586586
}
587587

588588
return this;
@@ -664,7 +664,7 @@ function finish(stream, state) {
664664
}
665665

666666
// TODO(ronag): Avoid using events to implement internal logic.
667-
function onFinished(stream, state, cb) {
667+
function onFinished(stream, cb) {
668668
function onerror(err) {
669669
stream.removeListener('finish', onfinish);
670670
stream.removeListener('error', onerror);

0 commit comments

Comments
 (0)