From bb79e768e5ab150f2075780734005783d53eb3ca Mon Sep 17 00:00:00 2001 From: Vladimir Ilic Date: Fri, 12 Oct 2018 11:09:51 -0700 Subject: [PATCH] lib: migrate from process.binding('config') to getOptions() PR-URL: https://github.com/nodejs/node/pull/23588 Reviewed-By: James M Snell Reviewed-By: Gireesh Punathil Reviewed-By: Sakthipriyan Vairamani Reviewed-By: Trivikram Kamat --- lib/internal/bootstrap/node.js | 18 +++++++++++------- 1 file changed, 11 insertions(+), 7 deletions(-) diff --git a/lib/internal/bootstrap/node.js b/lib/internal/bootstrap/node.js index bc87801be2769b..1ba427a572e3f7 100644 --- a/lib/internal/bootstrap/node.js +++ b/lib/internal/bootstrap/node.js @@ -112,13 +112,18 @@ NativeModule.require('internal/inspector_async_hook').setup(); } - const options = internalBinding('options'); - if (options.getOptions('--help')) { + const { getOptions } = internalBinding('options'); + const helpOption = getOptions('--help'); + const completionBashOption = getOptions('--completion-bash'); + const experimentalModulesOption = getOptions('--experimental-modules'); + const experimentalVMModulesOption = getOptions('--experimental-vm-modules'); + const experimentalWorkerOption = getOptions('--experimental-worker'); + if (helpOption) { NativeModule.require('internal/print_help').print(process.stdout); return; } - if (options.getOptions('--completion-bash')) { + if (completionBashOption) { NativeModule.require('internal/bash_completion').print(process.stdout); return; } @@ -138,7 +143,7 @@ setupQueueMicrotask(); } - if (process.binding('config').experimentalWorker) { + if (experimentalWorkerOption) { setupDOMException(); } @@ -170,9 +175,8 @@ 'DeprecationWarning', 'DEP0062', startup, true); } - if (process.binding('config').experimentalModules || - process.binding('config').experimentalVMModules) { - if (process.binding('config').experimentalModules) { + if (experimentalModulesOption || experimentalVMModulesOption) { + if (experimentalModulesOption) { process.emitWarning( 'The ESM module loader is experimental.', 'ExperimentalWarning', undefined);