Commit 49b5933
lib: simplify 'processChunkSync'
According to the real logic codes, it seems no matter whether 'nread >=
kMaxLength' or not. We always close the 'self' stream first. So we can
shorten it by merging them into one sample.
PR-URL: #22802
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Weijia Wang <starkwang@126.com>1 parent f0a4017 commit 49b5933
1 file changed
Lines changed: 1 addition & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -527,14 +527,12 @@ function processChunkSync(self, chunk, flushFlag) { | |||
| 527 | 527 | } | |
| 528 | 528 | ||
| 529 | 529 | self.bytesWritten = inputRead; | |
| 530 | + _close(self); | ||
| 530 | 531 | ||
| 531 | 532 | if (nread >= kMaxLength) { | |
| 532 | - _close(self); | ||
| 533 | 533 | throw new ERR_BUFFER_TOO_LARGE(); | |
| 534 | 534 | } | |
| 535 | 535 | ||
| 536 | - _close(self); | ||
| 537 | - | ||
| 538 | 536 | if (nread === 0) | |
| 539 | 537 | return Buffer.alloc(0); | |
| 540 | 538 | ||
0 commit comments