Commit 1425e75
stream: avoid function call where possible
Instead of assigning a boolean, move the function call that assigns a
value to the boolean to the only place that boolean is checked. This
avoids the function call in cases where it is not needed.
Refs: #41488 (review)
PR-URL: #41534
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Gerhard Stöbich <deb2001-github@yahoo.de>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>1 parent ec72cb4 commit 1425e75
1 file changed
Lines changed: 6 additions & 4 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -113,17 +113,19 @@ function isReadableFinished(stream, strict) { | |||
| 113 | 113 | ||
| 114 | 114 | function isReadable(stream) { | |
| 115 | 115 | if (stream && stream[kIsReadable] != null) return stream[kIsReadable]; | |
| 116 | - const r = isReadableNodeStream(stream); | ||
| 117 | 116 | if (typeof stream?.readable !== 'boolean') return null; | |
| 118 | 117 | if (isDestroyed(stream)) return false; | |
| 119 | - return r && stream.readable && !isReadableFinished(stream); | ||
| 118 | + return isReadableNodeStream(stream) && | ||
| 119 | + stream.readable && | ||
| 120 | + !isReadableFinished(stream); | ||
| 120 | 121 | } | |
| 121 | 122 | ||
| 122 | 123 | function isWritable(stream) { | |
| 123 | - const r = isWritableNodeStream(stream); | ||
| 124 | 124 | if (typeof stream?.writable !== 'boolean') return null; | |
| 125 | 125 | if (isDestroyed(stream)) return false; | |
| 126 | - return r && stream.writable && !isWritableEnded(stream); | ||
| 126 | + return isWritableNodeStream(stream) && | ||
| 127 | + stream.writable && | ||
| 128 | + !isWritableEnded(stream); | ||
| 127 | 129 | } | |
| 128 | 130 | ||
| 129 | 131 | function isFinished(stream, opts) { | |
0 commit comments