Skip to content

Commit b094b49

Browse files
mscdexevanlucas
authored andcommitted
http: reuse existing headers array for raw values
PR-URL: #6533 Reviewed-By: Matteo Collina <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Fedor Indutny <[email protected]> Reviewed-By: Benjamin Gruenbaum <[email protected]>
1 parent 064607b commit b094b49

File tree

1 file changed

+4
-8
lines changed

1 file changed

+4
-8
lines changed

lib/_http_incoming.js

+4-8
Original file line numberDiff line numberDiff line change
@@ -103,21 +103,17 @@ IncomingMessage.prototype.destroy = function destroy(error) {
103103
IncomingMessage.prototype._addHeaderLines = _addHeaderLines;
104104
function _addHeaderLines(headers, n) {
105105
if (headers && headers.length) {
106-
var raw, dest;
106+
var dest;
107107
if (this.complete) {
108-
raw = this.rawTrailers;
108+
this.rawTrailers = headers;
109109
dest = this.trailers;
110110
} else {
111-
raw = this.rawHeaders;
111+
this.rawHeaders = headers;
112112
dest = this.headers;
113113
}
114114

115115
for (var i = 0; i < n; i += 2) {
116-
var k = headers[i];
117-
var v = headers[i + 1];
118-
raw.push(k);
119-
raw.push(v);
120-
this._addHeaderLine(k, v, dest);
116+
this._addHeaderLine(headers[i], headers[i + 1], dest);
121117
}
122118
}
123119
}

0 commit comments

Comments
 (0)