lib: replace every Symbol.for by SymbolFor primordials · nodejs/node@39898a9 · GitHub

nodejs / node Public

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

• Code
• Issues 635
• Pull requests 755
• Actions
• Projects
• Security and quality
• Insights

Additional navigation options

Commit 39898a9

Browse filesBrowse the repository at this point in the historyBrowse files

Sebastien-Ahkrin

authored and

MylesBorins

committed

lib: replace every Symbol.for by SymbolFor primordials

PR-URL: #30857 Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>

1 parent 1cce000 commit 39898a9
Copy full SHA for 39898a9

6 files changed

+12-6Lines changed: 12 additions & 6 deletions

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

• lib
  • _http_server.js
  • _tls_wrap.js
  • events.js
  • internal
    • errors.js
    • util.js
    • worker.js

Expand file treeCollapse file tree

Open diff view settings

Collapse file

‎lib/_http_server.js‎

Copy file name to clipboardExpand all lines: lib/_http_server.js

+2-1Lines changed: 2 additions & 1 deletion

Original file line number	Diff line number	Diff line change
@@ -25,6 +25,7 @@ const {
25	25	ObjectKeys,
26	26	ObjectSetPrototypeOf,
27	27	Symbol,
	28	+ SymbolFor,
28	29	} = primordials;
29	30
30	31	const net = require('net');
@@ -378,7 +379,7 @@ Server.prototype[EE.captureRejectionSymbol] = function(
378	379	}
379	380	break;
380	381	default:
381		- net.Server.prototype[Symbol.for('nodejs.rejection')]
	382	+ net.Server.prototype[SymbolFor('nodejs.rejection')]
382	383	.call(this, err, event, ...args);
383	384	}
384	385	};

Collapse file

‎lib/_tls_wrap.js‎

Copy file name to clipboardExpand all lines: lib/_tls_wrap.js

+2-1Lines changed: 2 additions & 1 deletion

Original file line number	Diff line number	Diff line change
@@ -26,6 +26,7 @@ const {
26	26	ObjectDefineProperty,
27	27	ObjectSetPrototypeOf,
28	28	Symbol,
	29	+ SymbolFor,
29	30	} = primordials;
30	31
31	32	const {
@@ -1294,7 +1295,7 @@ Server.prototype[EE.captureRejectionSymbol] = function(
1294	1295	sock.destroy(err);
1295	1296	break;
1296	1297	default:
1297		- net.Server.prototype[Symbol.for('nodejs.rejection')]
	1298	+ net.Server.prototype[SymbolFor('nodejs.rejection')]
1298	1299	.call(this, err, event, sock);
1299	1300	}
1300	1301	};

Collapse file

‎lib/events.js‎

Copy file name to clipboardExpand all lines: lib/events.js

+2-1Lines changed: 2 additions & 1 deletion

Original file line number	Diff line number	Diff line change
@@ -33,8 +33,9 @@ const {
33	33	ReflectApply,
34	34	ReflectOwnKeys,
35	35	Symbol,
	36	+ SymbolFor,
36	37	} = primordials;
37		- const kRejection = Symbol.for('nodejs.rejection');
	38	+ const kRejection = SymbolFor('nodejs.rejection');
38	39
39	40	let spliceOne;
40	41

Collapse file

‎lib/internal/errors.js‎

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

+2-1Lines changed: 2 additions & 1 deletion

Original file line number	Diff line number	Diff line change
@@ -17,6 +17,7 @@ const {
17	17	ObjectDefineProperty,
18	18	ObjectKeys,
19	19	Symbol,
	20	+ SymbolFor,
20	21	} = primordials;
21	22
22	23	const messages = new Map();
@@ -199,7 +200,7 @@ class SystemError extends Error {
199	200	return `${this.name} [${this.code}]: ${this.message}`;
200	201	}
201	202
202		- [Symbol.for('nodejs.util.inspect.custom')](recurseTimes, ctx) {
	203	+ [SymbolFor('nodejs.util.inspect.custom')](recurseTimes, ctx) {
203	204	return lazyInternalUtilInspect().inspect(this, {
204	205	...ctx,
205	206	getters: true,

Collapse file

‎lib/internal/util.js‎

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

+2-1Lines changed: 2 additions & 1 deletion

Original file line number	Diff line number	Diff line change
@@ -12,6 +12,7 @@ const {
12	12	ObjectSetPrototypeOf,
13	13	ReflectConstruct,
14	14	Symbol,
	15	+ SymbolFor,
15	16	} = primordials;
16	17
17	18	const {
@@ -425,7 +426,7 @@ module.exports = {
425	426
426	427	// Symbol used to provide a custom inspect function for an object as an
427	428	// alternative to using 'inspect'
428		- customInspectSymbol: Symbol.for('nodejs.util.inspect.custom'),
	429	+ customInspectSymbol: SymbolFor('nodejs.util.inspect.custom'),
429	430
430	431	// Used by the buffer module to capture an internal reference to the
431	432	// default isEncoding implementation, just in case userland overrides it.

Collapse file

‎lib/internal/worker.js‎

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

+2-1Lines changed: 2 additions & 1 deletion

Original file line number	Diff line number	Diff line change
@@ -8,6 +8,7 @@ const {
8	8	ObjectCreate,
9	9	ObjectEntries,
10	10	Symbol,
	11	+ SymbolFor,
11	12	} = primordials;
12	13
13	14	const EventEmitter = require('events');
@@ -62,7 +63,7 @@ const kOnCouldNotSerializeErr = Symbol('kOnCouldNotSerializeErr');
62	63	const kOnErrorMessage = Symbol('kOnErrorMessage');
63	64	const kParentSideStdio = Symbol('kParentSideStdio');
64	65
65		- const SHARE_ENV = Symbol.for('nodejs.worker_threads.SHARE_ENV');
	66	+ const SHARE_ENV = SymbolFor('nodejs.worker_threads.SHARE_ENV');
66	67	const debug = require('internal/util/debuglog').debuglog('worker');
67	68
68	69	let cwdCounter;

0 commit comments

Comments

0 (0)