Skip to content

Commit 8da4143

Browse files
bnoordhuisgibfahn
authored andcommitted
http: remove adapter frame from onParserExecute
Remove a pointless adapter frame by fixing up the function's formal parameter count. Before: frame #0: 0x000033257ea446d5 onParserExecute(...) frame #1: 0x000033257ea3b93f <adaptor> frame #2: 0x000033257ea41959 <internal> frame #3: 0x000033257e9840ff <entry> After: frame #0: 0x00000956287446d5 onParserExecute(...) frame #1: 0x0000095628741959 <internal> frame #2: 0x00000956286840ff <entry> PR-URL: #17693 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Daniel Bevenius <daniel.bevenius@gmail.com> Reviewed-By: Khaidi Chu <i@2333.moe>
1 parent 70daf95 commit 8da4143

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

lib/_http_server.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -456,7 +456,7 @@ function socketOnData(server, socket, parser, state, d) {
456456
onParserExecuteCommon(server, socket, parser, state, ret, d);
457457
}
458458

459-
function onParserExecute(server, socket, parser, state, ret, d) {
459+
function onParserExecute(server, socket, parser, state, ret) {
460460
socket._unrefTimer();
461461
debug('SERVER socketOnParserExecute %d', ret);
462462
onParserExecuteCommon(server, socket, parser, state, ret, undefined);

0 commit comments

Comments
 (0)