http2: remove no longer userful options.selectPadding · nodejs/node@d3d662a · GitHub

nodejs / node Public

• Notifications You must be signed in to change notification settings
• Fork 36.6k
• Star 120k

• Code
• Issues 633
• Pull requests 748
• Actions
• Projects
• Security and quality
• Insights

Additional navigation options

Commit d3d662a

Browse filesBrowse the repository at this point in the historyBrowse files

hkleungai

authored and

aduh95

committed

http2: remove no longer userful options.selectPadding

This is a followup cleanup on a previous PR. Current source code and doc explicitly states that `selectPadding` has been removed for all exports. Refs: #29144 Refs: https://nodejs.org/api/http2.html PR-URL: #58373 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>

1 parent 98b6aa0 commit d3d662a
Copy full SHA for d3d662a

1 file changed

-3Lines changed: 0 additions & 3 deletions

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

• lib/internal/http2
  • core.js

Expand file treeCollapse file tree

Open diff view settings

Collapse file

‎lib/internal/http2/core.js‎

Copy file name to clipboardExpand all lines: lib/internal/http2/core.js

-3Lines changed: 0 additions & 3 deletions

Original file line number	Diff line number	Diff line change
@@ -254,7 +254,6 @@ const kProceed = Symbol('proceed');
254	254	const kProtocol = Symbol('protocol');
255	255	const kRemoteSettings = Symbol('remote-settings');
256	256	const kRequestAsyncResource = Symbol('requestAsyncResource');
257		- const kSelectPadding = Symbol('select-padding');
258	257	const kSentHeaders = Symbol('sent-headers');
259	258	const kSentTrailers = Symbol('sent-trailers');
260	259	const kServer = Symbol('server');
@@ -1108,8 +1107,6 @@ function setupHandle(socket, type, options) {
1108	1107	const handle = new binding.Http2Session(type);
1109	1108	handle[kOwner] = this;
1110	1109
1111		- if (typeof options.selectPadding === 'function')
1112		- this[kSelectPadding] = options.selectPadding;
1113	1110	handle.consume(socket._handle);
1114	1111	handle.ongracefulclosecomplete = this[kMaybeDestroy].bind(this, null);
1115	1112

0 commit comments

Comments

0 (0)