http2: order declarations in core.js

Order declarations: * public modules in alphabetical order * internal modules in alphabetical order * process.binding() calls in alphabetical order * exports in alphabetical order PR-URL: #21689 Reviewed-By: Gus Caplan <[email protected]> Reviewed-By: Weijia Wang <[email protected]> Reviewed-By: James M Snell <[email protected]>

Author: Trott

lib/internal/http2/core.js

@@ -2,22 +2,31 @@
 
 /* eslint-disable no-use-before-define */
 
-constinternalUtil=require('internal/util');
-internalUtil.assertCrypto();
+const{
+ assertCrypto,
+customInspectSymbol: kInspect,
+ promisify
+}=require('internal/util');
+
+assertCrypto();
 
-const{ internalBinding }=require('internal/bootstrap/loaders');
-const{ async_id_symbol }=require('internal/async_hooks').symbols;
-const{UV_EOF}=process.binding('uv');
-consthttp=require('http');
-constbinding=process.binding('http2');
-const{ FileHandle }=process.binding('fs');
-const{ StreamPipe }=internalBinding('stream_pipe');
 constassert=require('assert');
 constEventEmitter=require('events');
+constfs=require('fs');
+consthttp=require('http');
 constnet=require('net');
+const{ Duplex }=require('stream');
 consttls=require('tls');
+const{URL}=require('url');
 constutil=require('util');
-constfs=require('fs');
+
+const{ kIncomingMessage }=require('_http_common');
+const{ kServerResponse }=require('_http_server');
+const{ StreamWrap }=require('_stream_wrap');
+
+const{ defaultTriggerAsyncIdScope }=require('internal/async_hooks');
+const{ async_id_symbol }=require('internal/async_hooks').symbols;
+const{ internalBinding }=require('internal/bootstrap/loaders');
 const{
 codes: {
 ERR_HTTP2_ALTSVC_INVALID_ORIGIN,
@@ -61,30 +70,11 @@ const{
 ERR_SOCKET_CLOSED
 }
 }=require('internal/errors');
-const{ StreamWrap }=require('_stream_wrap');
-const{ Duplex }=require('stream');
-const{URL}=require('url');
+const{ utcDate }=require('internal/http');
 const{ onServerStream,
  Http2ServerRequest,
  Http2ServerResponse,
 }=require('internal/http2/compat');
-const{ utcDate }=require('internal/http');
-const{
- promisify,
-customInspectSymbol: kInspect
-}=internalUtil;
-const{ isArrayBufferView }=require('internal/util/types');
-const{ defaultTriggerAsyncIdScope }=require('internal/async_hooks');
-const{_connectionListener: httpConnectionListener}=http;
-const{ createPromise, promiseResolve }=process.binding('util');
-constdebug=util.debuglog('http2');
-
-constkMaxFrameSize=(2**24)-1;
-constkMaxInt=(2**32)-1;
-constkMaxStreams=(2**31)-1;
-
-// eslint-disable-next-line no-control-regex
-constkQuotedString=/^[\x09\x20-\x5b\x5d-\x7e\x80-\xff]*$/;
 
 const{
  assertIsObject,
@@ -104,27 +94,40 @@ const{
  updateOptionsBuffer,
  updateSettingsBuffer
 }=require('internal/http2/util');
-
-const{
- kTimeout,
- setUnrefTimeout,
- validateTimerDuration
-}=require('internal/timers');
 const{
  createWriteWrap,
  writeGeneric,
  writevGeneric
 }=require('internal/stream_base_commons');
+const{
+ kTimeout,
+ setUnrefTimeout,
+ validateTimerDuration
+}=require('internal/timers');
+const{ isArrayBufferView }=require('internal/util/types');
 
+const{ FileHandle }=process.binding('fs');
+constbinding=process.binding('http2');
 const{ ShutdownWrap }=process.binding('stream_wrap');
+const{ createPromise, promiseResolve }=process.binding('util');
+const{UV_EOF}=process.binding('uv');
+
+const{ StreamPipe }=internalBinding('stream_pipe');
+const{_connectionListener: httpConnectionListener}=http;
+constdebug=util.debuglog('http2');
+
+constkMaxFrameSize=(2**24)-1;
+constkMaxInt=(2**32)-1;
+constkMaxStreams=(2**31)-1;
+
+// eslint-disable-next-line no-control-regex
+constkQuotedString=/^[\x09\x20-\x5b\x5d-\x7e\x80-\xff]*$/;
+
 const{ constants, nameForErrorCode }=binding;
 
 constNETServer=net.Server;
 constTLSServer=tls.Server;
 
-const{ kIncomingMessage }=require('_http_common');
-const{ kServerResponse }=require('_http_server');
-
 constkAlpnProtocol=Symbol('alpnProtocol');
 constkAuthority=Symbol('authority');
 constkEncrypted=Symbol('encrypted');
@@ -2815,13 +2818,13 @@ function getUnpackedSettings(buf, options ={}){
 
 // Exports
 module.exports={
+ connect,
  constants,
+ createServer,
+ createSecureServer,
  getDefaultSettings,
  getPackedSettings,
  getUnpackedSettings,
- createServer,
- createSecureServer,
- connect,
  Http2Session,
  Http2Stream,
  Http2ServerRequest,