Commit c21f1f0
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 <luigipinca@gmail.com>
Reviewed-By: Gerhard Stöbich <deb2001-github@yahoo.de>1 parent 4b6aa07 commit c21f1f0
1 file changed
Lines changed: 2 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -582,7 +582,7 @@ Writable.prototype.end = function(chunk, encoding, cb) { | |||
| 582 | 582 | if (err || state.finished) | |
| 583 | 583 | process.nextTick(cb, err); | |
| 584 | 584 | else | |
| 585 | - onFinished(this, state, cb); | ||
| 585 | + onFinished(this, cb); | ||
| 586 | 586 | } | |
| 587 | 587 | ||
| 588 | 588 | return this; | |
@@ -664,7 +664,7 @@ function finish(stream, state) { | |||
| 664 | 664 | } | |
| 665 | 665 | ||
| 666 | 666 | // TODO(ronag): Avoid using events to implement internal logic. | |
| 667 | - function onFinished(stream, state, cb) { | ||
| 667 | + function onFinished(stream, cb) { | ||
| 668 | 668 | function onerror(err) { | |
| 669 | 669 | stream.removeListener('finish', onfinish); | |
| 670 | 670 | stream.removeListener('error', onerror); | |
0 commit comments