From 8da41434cf196e8018a7f44d6370ce7af8bc342f Mon Sep 17 00:00:00 2001 From: Ben Noordhuis Date: Fri, 15 Dec 2017 04:12:00 +0100 Subject: [PATCH] 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 frame #2: 0x000033257ea41959 frame #3: 0x000033257e9840ff After: frame #0: 0x00000956287446d5 onParserExecute(...) frame #1: 0x0000095628741959 frame #2: 0x00000956286840ff PR-URL: https://github.com/nodejs/node/pull/17693 Reviewed-By: Anna Henningsen Reviewed-By: Colin Ihrig Reviewed-By: James M Snell Reviewed-By: Daniel Bevenius Reviewed-By: Khaidi Chu --- lib/_http_server.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/_http_server.js b/lib/_http_server.js index dcfea2f1b17..04b6ef75302 100644 --- a/lib/_http_server.js +++ b/lib/_http_server.js @@ -456,7 +456,7 @@ function socketOnData(server, socket, parser, state, d) { onParserExecuteCommon(server, socket, parser, state, ret, d); } -function onParserExecute(server, socket, parser, state, ret, d) { +function onParserExecute(server, socket, parser, state, ret) { socket._unrefTimer(); debug('SERVER socketOnParserExecute %d', ret); onParserExecuteCommon(server, socket, parser, state, ret, undefined);