diff options
author | cjihrig <cjihrig@gmail.com> | 2018-12-02 12:08:26 -0500 |
---|---|---|
committer | Rich Trott <rtrott@gmail.com> | 2018-12-04 11:32:42 -0800 |
commit | 7ac939e106bf8d878f6e193bfef317f9bd237551 (patch) | |
tree | 3746e924074e01e85832174e0329a2a68f6d1545 /lib | |
parent | 585380d2333e95763a9e8adea57c272297b15e8b (diff) | |
download | android-node-v8-7ac939e106bf8d878f6e193bfef317f9bd237551.tar.gz android-node-v8-7ac939e106bf8d878f6e193bfef317f9bd237551.tar.bz2 android-node-v8-7ac939e106bf8d878f6e193bfef317f9bd237551.zip |
fs: simplify fs.promises warning logic
fs.promises is lazy loaded. Instead of using a seaparate
Boolean flag to track whether or not it has been loaded, just
inspect the state of the lazy loaded module itself.
PR-URL: https://github.com/nodejs/node/pull/24788
Reviewed-By: Anto Aravinth <anto.aravinth.cse@gmail.com>
Reviewed-By: Weijia Wang <starkwang@126.com>
Reviewed-By: Masashi Hirano <shisama07@gmail.com>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
Diffstat (limited to 'lib')
-rw-r--r-- | lib/fs.js | 6 |
1 files changed, 2 insertions, 4 deletions
@@ -86,12 +86,11 @@ const { validateUint32 } = require('internal/validators'); -let promisesWarn = true; let truncateWarn = true; let fs; // Lazy loaded -let promises; +let promises = null; let watchers; let ReadFileContext; let ReadStream; @@ -1931,9 +1930,8 @@ Object.defineProperties(fs, { configurable: true, enumerable: false, get() { - if (promisesWarn) { + if (promises === null) { promises = require('internal/fs/promises'); - promisesWarn = false; process.emitWarning('The fs.promises API is experimental', 'ExperimentalWarning'); } |