From eb0101925c3926fcd6a8b6256cf5cf89a8c73270 Mon Sep 17 00:00:00 2001 From: Efe Karasakal Date: Wed, 8 Jul 2026 20:04:38 +0200 Subject: [PATCH] http: remove unused n arg from IncomingMessage._read Signed-off-by: Efe Karasakal --- lib/_http_incoming.js | 10 +--------- 1 file changed, 1 insertion(+), 9 deletions(-) diff --git a/lib/_http_incoming.js b/lib/_http_incoming.js index 6c082454b08a2b..598e1a267aa113 100644 --- a/lib/_http_incoming.js +++ b/lib/_http_incoming.js @@ -214,15 +214,7 @@ IncomingMessage.prototype.setTimeout = function setTimeout(msecs, callback) { return this; }; -// Argument n cannot be factored out due to the overhead of -// 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) { +IncomingMessage.prototype._read = function _read() { if (!this._consuming) { this._readableState.readingMore = false; this._consuming = true;