summaryrefslogtreecommitdiff
path: root/lib
diff options
context:
space:
mode:
authorMyles Borins <mylesborins@google.com>2019-03-28 01:21:15 -0400
committerMichaël Zasso <targos@protonmail.com>2019-03-30 13:02:33 +0100
commit7a547098d5b13e14b8ad86145fd889fb18b7f3d4 (patch)
tree8dc6ce458b567db06176b684972323b9299ac819 /lib
parent6bc2ab19e9f338f99de259a7e3f4c9436fd1b9fc (diff)
downloadandroid-node-v8-7a547098d5b13e14b8ad86145fd889fb18b7f3d4.tar.gz
android-node-v8-7a547098d5b13e14b8ad86145fd889fb18b7f3d4.tar.bz2
android-node-v8-7a547098d5b13e14b8ad86145fd889fb18b7f3d4.zip
esm: use primordials
Converted uses of Object and Map to use frozen built in primordials. PR-URL: https://github.com/nodejs/node/pull/26954 Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: Guy Bedford <guybedford@gmail.com> Reviewed-By: Gus Caplan <me@gus.host> Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Diffstat (limited to 'lib')
-rw-r--r--lib/internal/modules/esm/default_resolve.js7
-rw-r--r--lib/internal/modules/esm/loader.js7
-rw-r--r--lib/internal/modules/esm/translators.js1
3 files changed, 13 insertions, 2 deletions
diff --git a/lib/internal/modules/esm/default_resolve.js b/lib/internal/modules/esm/default_resolve.js
index 8da24cf5b3..8b8c5a2b5a 100644
--- a/lib/internal/modules/esm/default_resolve.js
+++ b/lib/internal/modules/esm/default_resolve.js
@@ -17,7 +17,12 @@ const { pathToFileURL, fileURLToPath } = require('internal/url');
const { ERR_ENTRY_TYPE_MISMATCH,
ERR_UNKNOWN_FILE_EXTENSION } = require('internal/errors').codes;
-const realpathCache = new Map();
+const {
+ Object,
+ SafeMap
+} = primordials;
+
+const realpathCache = new SafeMap();
// const TYPE_NONE = 0;
const TYPE_COMMONJS = 1;
diff --git a/lib/internal/modules/esm/loader.js b/lib/internal/modules/esm/loader.js
index 465775f56f..88f03119fc 100644
--- a/lib/internal/modules/esm/loader.js
+++ b/lib/internal/modules/esm/loader.js
@@ -22,6 +22,11 @@ const FunctionBind = Function.call.bind(Function.prototype.bind);
const debug = require('internal/util/debuglog').debuglog('esm');
+const {
+ Object,
+ SafeMap
+} = primordials;
+
/* A Loader instance is used as the main entry point for loading ES modules.
* Currently, this is a singleton -- there is only one used for loading
* the main module and everything in its dependency graph. */
@@ -35,7 +40,7 @@ class Loader {
this.moduleMap = new ModuleMap();
// Map of already-loaded CJS modules to use
- this.cjsCache = new Map();
+ this.cjsCache = new SafeMap();
// The resolver has the signature
// (specifier : string, parentURL : string, defaultResolve)
diff --git a/lib/internal/modules/esm/translators.js b/lib/internal/modules/esm/translators.js
index 172569fb04..4b2db024c5 100644
--- a/lib/internal/modules/esm/translators.js
+++ b/lib/internal/modules/esm/translators.js
@@ -13,6 +13,7 @@ const createDynamicModule = require(
const fs = require('fs');
const {
SafeMap,
+ JSON
} = primordials;
const { fileURLToPath, URL } = require('url');
const { debuglog } = require('internal/util/debuglog');