diff options
author | Myles Borins <mylesborins@google.com> | 2019-03-28 01:21:15 -0400 |
---|---|---|
committer | Michaël Zasso <targos@protonmail.com> | 2019-03-30 13:02:33 +0100 |
commit | 7a547098d5b13e14b8ad86145fd889fb18b7f3d4 (patch) | |
tree | 8dc6ce458b567db06176b684972323b9299ac819 /lib | |
parent | 6bc2ab19e9f338f99de259a7e3f4c9436fd1b9fc (diff) | |
download | android-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.js | 7 | ||||
-rw-r--r-- | lib/internal/modules/esm/loader.js | 7 | ||||
-rw-r--r-- | lib/internal/modules/esm/translators.js | 1 |
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'); |