Commit c846184
fs: remove needless assignment of null
This line `pool = null;` isn't needed and has
been around since the first iteration of streams.
I can't find a good reason for it to exist, it's
not more readable, nor does it seem to trick the
compiler into any optimizations.
PR-URL: #10260
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Italo A. Casas <me@italoacasas.com>1 parent 6ad8a6e commit c846184
1 file changed
Lines changed: 0 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -1961,7 +1961,6 @@ ReadStream.prototype._read = function(n) { | |||
| 1961 | 1961 | ||
| 1962 | 1962 | if (!pool || pool.length - pool.used < kMinPoolSpace) { | |
| 1963 | 1963 | // discard the old pool. | |
| 1964 | - pool = null; | ||
| 1965 | 1964 | allocNewPool(this._readableState.highWaterMark); | |
| 1966 | 1965 | } | |
| 1967 | 1966 | ||
0 commit comments