diff options
| author | Robert Nagy <ronagy@icloud.com> | 2019-10-20 10:13:57 +0200 |
|---|---|---|
| committer | Rich Trott <rtrott@gmail.com> | 2019-11-11 19:50:40 -0800 |
| commit | f8c069f5b88a25304ee2fc638c51464b4df196dd (patch) | |
| tree | 66de4cb08640930e92c0538cf30a6cc5bca06b3f /src/node_contextify.cc | |
| parent | e55d65384d3feb3dd1bb7341a23766663fbcdba6 (diff) | |
| download | android-node-v8-f8c069f5b88a25304ee2fc638c51464b4df196dd.tar.gz android-node-v8-f8c069f5b88a25304ee2fc638c51464b4df196dd.tar.bz2 android-node-v8-f8c069f5b88a25304ee2fc638c51464b4df196dd.zip | |
stream: increase MAX_HWM
MAX_HWM was added in 9208c89 where the highwatermark was changed to
always increase in steps of highest power of 2 to prevent increasing
hwm excessivly in tiny amounts.
Why a limit was added on the highwatermark is unclear but breaks
existing usage where a larger read size is used. The invariant for
read(n) is that a buffer of size n is always returned. Considering
a maximum ceiling on the buffer size breaks this invariant.
This PR significantly increases the limit to make it less likely to
break the previous invariant and also documents the limit.
Fixes: https://github.com/nodejs/node/issues/29933
PR-URL: https://github.com/nodejs/node/pull/29938
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Diffstat (limited to 'src/node_contextify.cc')
0 files changed, 0 insertions, 0 deletions