Commit 560f657
stream: delete redundant code
In `Writable.prototype.end()`, `state.ending` is true after calling
`endWritable()` and it doesn't reset to false.
In `Writable.prototype.uncork()`, `state.finished` must be false
if `state.bufferedRequest` is true.
PR-URL: #18145
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>1 parent 58003d4 commit 560f657
1 file changed
Lines changed: 1 addition & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -307,7 +307,6 @@ Writable.prototype.uncork = function() { | |||
| 307 | 307 | ||
| 308 | 308 | if (!state.writing && | |
| 309 | 309 | !state.corked && | |
| 310 | - !state.finished && | ||
| 311 | 310 | !state.bufferProcessing && | |
| 312 | 311 | state.bufferedRequest) | |
| 313 | 312 | clearBuffer(this, state); | |
@@ -579,7 +578,7 @@ Writable.prototype.end = function(chunk, encoding, cb) { | |||
| 579 | 578 | } | |
| 580 | 579 | ||
| 581 | 580 | // ignore unnecessary end() calls. | |
| 582 | - if (!state.ending && !state.finished) | ||
| 581 | + if (!state.ending) | ||
| 583 | 582 | endWritable(this, state, cb); | |
| 584 | 583 | }; | |
| 585 | 584 | ||
0 commit comments