diff options
author | Qingyu Deng <bitdqy@hotmail.com> | 2021-02-10 08:10:36 +0300 |
---|---|---|
committer | Daniele Belardi <dwon.dnl@gmail.com> | 2021-02-10 14:30:16 +0300 |
commit | 9d2125e3bfeaa36692ded547a50c9579ddc39e20 (patch) | |
tree | b91cff9d0caee2889580cd96ee17d99b6250d707 | |
parent | 9fa1f68b55397799cf59505dd188039a06566b36 (diff) |
http: explain the possibilty of refactor unused argument
PR-URL: https://github.com/nodejs/node/pull/37275
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
-rw-r--r-- | lib/_http_incoming.js | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/lib/_http_incoming.js b/lib/_http_incoming.js index 8a95d7df4af..30432a5230f 100644 --- a/lib/_http_incoming.js +++ b/lib/_http_incoming.js @@ -155,6 +155,10 @@ IncomingMessage.prototype.setTimeout = function setTimeout(msecs, callback) { // argument adaptor frame creation inside V8 in case that number of actual // arguments is different from expected arguments. // Ref: https://bugs.chromium.org/p/v8/issues/detail?id=10201 +// NOTE: Argument adapt frame issue might be solved in V8 engine v8.9. +// Refactoring `n` out might be possible when V8 is upgraded to that +// version. +// Ref: https://v8.dev/blog/v8-release-89 IncomingMessage.prototype._read = function _read(n) { if (!this._consuming) { this._readableState.readingMore = false; |