Skip to content

Commit 9306de2

Browse files
bnoordhuisMylesBorins
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 <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Daniel Bevenius <[email protected]> Reviewed-By: Khaidi Chu <[email protected]>
1 parent af1b340 commit 9306de2

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
@@ -449,7 +449,7 @@ function socketOnData(server, socket, parser, state, d) {
449449
onParserExecuteCommon(server, socket, parser, state, ret, d);
450450
}
451451

452-
function onParserExecute(server, socket, parser, state, ret, d) {
452+
function onParserExecute(server, socket, parser, state, ret) {
453453
socket._unrefTimer();
454454
debug('SERVER socketOnParserExecute %d', ret);
455455
onParserExecuteCommon(server, socket, parser, state, ret, undefined);

0 commit comments

Comments
 (0)