Skip to content

Commit 5df3874

Browse files
committed
http: remove unused n arg from IncomingMessage._read
1 parent 799e910 commit 5df3874

1 file changed

Lines changed: 1 addition & 9 deletions

File tree

lib/_http_incoming.js

Lines changed: 1 addition & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -214,15 +214,7 @@ IncomingMessage.prototype.setTimeout = function setTimeout(msecs, callback) {
214214
return this;
215215
};
216216

217-
// Argument n cannot be factored out due to the overhead of
218-
// argument adaptor frame creation inside V8 in case that number of actual
219-
// arguments is different from expected arguments.
220-
// Ref: https://bugs.chromium.org/p/v8/issues/detail?id=10201
221-
// NOTE: Argument adapt frame issue might be solved in V8 engine v8.9.
222-
// Refactoring `n` out might be possible when V8 is upgraded to that
223-
// version.
224-
// Ref: https://v8.dev/blog/v8-release-89
225-
IncomingMessage.prototype._read = function _read(n) {
217+
IncomingMessage.prototype._read = function _read() {
226218
if (!this._consuming) {
227219
this._readableState.readingMore = false;
228220
this._consuming = true;

0 commit comments

Comments
 (0)