lib: replace Set global by the primordials

PR-URL: #31154 Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: David Carlier <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Trivikram Kamat <[email protected]>
nodejs · Jan 6, 2020 · 3f21ad6 · 3f21ad6
1 parent 542aae4
commit 3f21ad6
Show file tree

Hide file tree

Showing 15 changed files with 19 additions and 0 deletions.
diff --git a/lib/.eslintrc.yaml b/lib/.eslintrc.yaml
@@ -27,6 +27,8 @@ rules:
       message: "Use `const { Promise } = primordials;` instead of the global."
     - name: Reflect
       message: "Use `const { Reflect } = primordials;` instead of the global."
+    - name: Set
+      message: "Use `const { Set } = primordials;` instead of the global."
     - name: Symbol
       message: "Use `const { Symbol } = primordials;` instead of the global."
     - name: WeakMap

diff --git a/lib/_stream_readable.js b/lib/_stream_readable.js
@@ -27,6 +27,7 @@ const {
   NumberIsNaN,
   ObjectDefineProperties,
   ObjectSetPrototypeOf,
+  Set,
   SymbolAsyncIterator,
   Symbol
 } = primordials;

diff --git a/lib/internal/fs/rimraf.js b/lib/internal/fs/rimraf.js
@@ -8,6 +8,7 @@
 
 const {
   Promise,
+  Set,
 } = primordials;
 
 const { Buffer } = require('buffer');

diff --git a/lib/internal/http2/core.js b/lib/internal/http2/core.js
@@ -12,6 +12,7 @@ const {
   ObjectPrototypeHasOwnProperty,
   Promise,
   ReflectGetPrototypeOf,
+  Set,
   Symbol,
 } = primordials;
 

diff --git a/lib/internal/http2/util.js b/lib/internal/http2/util.js
@@ -6,6 +6,7 @@ const {
   Number,
   ObjectCreate,
   ObjectKeys,
+  Set,
   Symbol,
 } = primordials;
 

diff --git a/lib/internal/inspector_async_hook.js b/lib/internal/inspector_async_hook.js
@@ -3,6 +3,10 @@
 let hook;
 let config;
 
+const {
+  Set,
+} = primordials;
+
 function lazyHookCreation() {
   const inspector = internalBinding('inspector');
   const { createHook } = require('async_hooks');

diff --git a/lib/internal/modules/esm/create_dynamic_module.js b/lib/internal/modules/esm/create_dynamic_module.js
@@ -5,6 +5,7 @@ const {
   ArrayPrototypeMap,
   JSONStringify,
   ObjectCreate,
+  Set,
 } = primordials;
 
 const debug = require('internal/util/debuglog').debuglog('esm');

diff --git a/lib/internal/process/per_thread.js b/lib/internal/process/per_thread.js
@@ -12,6 +12,7 @@ const {
   ObjectFreeze,
   ObjectGetOwnPropertyDescriptors,
   RegExpPrototypeTest,
+  Set,
   SetPrototype,
   SetPrototypeHas,
   StringPrototypeReplace,

diff --git a/lib/internal/repl/utils.js b/lib/internal/repl/utils.js
@@ -2,6 +2,7 @@
 
 const {
   MathMin,
+  Set,
   Symbol,
 } = primordials;
 

diff --git a/lib/internal/util.js b/lib/internal/util.js
@@ -12,6 +12,7 @@ const {
   ObjectSetPrototypeOf,
   Promise,
   ReflectConstruct,
+  Set,
   Symbol,
   SymbolFor,
 } = primordials;

diff --git a/lib/internal/util/comparisons.js b/lib/internal/util/comparisons.js
@@ -14,6 +14,7 @@ const {
   ObjectPrototypeHasOwnProperty,
   ObjectPrototypePropertyIsEnumerable,
   ObjectPrototypeToString,
+  Set,
   StringPrototypeValueOf,
   SymbolPrototypeValueOf,
   SymbolToStringTag,

diff --git a/lib/internal/util/inspect.js b/lib/internal/util/inspect.js
@@ -33,6 +33,7 @@ const {
   ObjectPrototypePropertyIsEnumerable,
   ObjectSeal,
   RegExpPrototypeToString,
+  Set,
   SetPrototype,
   SetPrototypeValues,
   StringPrototypeValueOf,

diff --git a/lib/perf_hooks.js b/lib/perf_hooks.js
@@ -7,6 +7,7 @@ const {
   ObjectDefineProperties,
   ObjectDefineProperty,
   ObjectKeys,
+  Set,
   Symbol,
 } = primordials;
 

diff --git a/lib/repl.js b/lib/repl.js
@@ -55,6 +55,7 @@ const {
   ObjectSetPrototypeOf,
   Promise,
   PromiseRace,
+  Set,
   Symbol,
   WeakSet,
 } = primordials;

diff --git a/lib/trace_events.js b/lib/trace_events.js
@@ -2,6 +2,7 @@
 
 const {
   ArrayIsArray,
+  Set,
   Symbol,
 } = primordials;