Skip to content

Commit

Permalink
http2: reduce require calls in http2/core
Browse files Browse the repository at this point in the history
This commit removes unnecesary requires of http and internal/util in
http2/core.js

PR-URL: nodejs#20422
Reviewed-By: Anatoli Papirovski <apapirovski@mac.com>
Reviewed-By: Tiancheng "Timothy" Gu <timothygu99@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
  • Loading branch information
danbev authored and kjin committed Oct 16, 2018
1 parent c9865ff commit d9efc87
Showing 1 changed file with 5 additions and 3 deletions.
8 changes: 5 additions & 3 deletions lib/internal/http2/core.js
Original file line number Diff line number Diff line change
Expand Up @@ -23,9 +23,12 @@ const { onServerStream,
Http2ServerResponse,
} = require('internal/http2/compat');
const { utcDate } = require('internal/http');
const { promisify } = require('internal/util');
const {
promisify,
customInspectSymbol: kInspect
} = require('internal/util');
const { isArrayBufferView } = require('internal/util/types');
const { _connectionListener: httpConnectionListener } = require('http');
const { _connectionListener: httpConnectionListener } = http;
const { createPromise, promiseResolve } = process.binding('util');
const debug = util.debuglog('http2');

Expand Down Expand Up @@ -67,7 +70,6 @@ const { constants, nameForErrorCode } = binding;
const NETServer = net.Server;
const TLSServer = tls.Server;

const kInspect = require('internal/util').customInspectSymbol;
const { kIncomingMessage } = require('_http_common');
const { kServerResponse } = require('_http_server');

Expand Down

0 comments on commit d9efc87

Please sign in to comment.