From 9d2125e3bfeaa36692ded547a50c9579ddc39e20 Mon Sep 17 00:00:00 2001 From: Qingyu Deng Date: Wed, 10 Feb 2021 13:10:36 +0800 Subject: [PATCH] http: explain the possibilty of refactor unused argument PR-URL: https://github.com/nodejs/node/pull/37275 Reviewed-By: Antoine du Hamel Reviewed-By: Darshan Sen Reviewed-By: Matteo Collina --- lib/_http_incoming.js | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/lib/_http_incoming.js b/lib/_http_incoming.js index 8a95d7df4afaa8..30432a5230fa30 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;