GeezerVison is showing github.com — worker: remove `--experimental-worker` flag · nodejs/node@ab7284a
File treeExpand file treeCollapse file tree Open diff view settings Expand file treeCollapse file tree Open diff view settings Collapse file Copy file name to clipboardExpand all lines: lib/internal/bootstrap/loaders.js +1-2Lines changed: 1 addition & 2 deletions | Original file line number | Diff line number | Diff line change |
|---|
@@ -255,8 +255,7 @@ if (config.exposeInternals) { | 255 | 255 | }; | 256 | 256 |
| 257 | 257 | NativeModule.isInternal = function(id) { | 258 | | - return id.startsWith('internal/') || | 259 | | - (id === 'worker_threads' && !config.experimentalWorker); | | 258 | + return id.startsWith('internal/'); | 260 | 259 | }; | 261 | 260 | } | 262 | 261 |
| |
Collapse file Copy file name to clipboardExpand all lines: lib/internal/bootstrap/node.js +1-3Lines changed: 1 addition & 3 deletions | Original file line number | Diff line number | Diff line change |
|---|
@@ -179,9 +179,7 @@ function startup() { | 179 | 179 | setupQueueMicrotask(); | 180 | 180 | } | 181 | 181 |
| 182 | | - if (getOptionValue('--experimental-worker')) { | 183 | | - setupDOMException(); | 184 | | - } | | 182 | + setupDOMException(); | 185 | 183 |
| 186 | 184 | // On OpenBSD process.execPath will be relative unless we | 187 | 185 | // get the full path before process.execPath is used. | |
Collapse file Copy file name to clipboardExpand all lines: lib/internal/modules/cjs/helpers.js +1-8Lines changed: 1 addition & 8 deletions | Original file line number | Diff line number | Diff line change |
|---|
| 9 | 9 | CHAR_HASH, | 10 | 10 | } = require('internal/constants'); | 11 | 11 |
| 12 | | -const { getOptionValue } = require('internal/options'); | 13 | | - | 14 | 12 | // Invoke with makeRequireFunction(module) where |module| is the Module object | 15 | 13 | // to use as the context for the require() function. | 16 | 14 | function makeRequireFunction(mod) { | @@ -100,14 +98,9 @@ const builtinLibs = [ | 100 | 98 | 'dgram', 'dns', 'domain', 'events', 'fs', 'http', 'http2', 'https', 'net', | 101 | 99 | 'os', 'path', 'perf_hooks', 'punycode', 'querystring', 'readline', 'repl', | 102 | 100 | 'stream', 'string_decoder', 'tls', 'trace_events', 'tty', 'url', 'util', | 103 | | - 'v8', 'vm', 'zlib' | | 101 | + 'v8', 'vm', 'worker_threads', 'zlib' | 104 | 102 | ]; | 105 | 103 |
| 106 | | -if (getOptionValue('--experimental-worker')) { | 107 | | - builtinLibs.push('worker_threads'); | 108 | | - builtinLibs.sort(); | 109 | | -} | 110 | | - | 111 | 104 | if (typeof internalBinding('inspector').open === 'function') { | 112 | 105 | builtinLibs.push('inspector'); | 113 | 106 | builtinLibs.sort(); | |
Collapse file Copy file name to clipboardExpand all lines: src/node_config.cc -3Lines changed: 0 additions & 3 deletions | Original file line number | Diff line number | Diff line change |
|---|
@@ -89,9 +89,6 @@ static void Initialize(Local<Object> target, | 89 | 89 | if (env->options()->experimental_vm_modules) | 90 | 90 | READONLY_TRUE_PROPERTY(target, "experimentalVMModules"); | 91 | 91 |
| 92 | | - if (env->options()->experimental_worker) | 93 | | - READONLY_TRUE_PROPERTY(target, "experimentalWorker"); | 94 | | - | 95 | 92 | if (env->options()->experimental_repl_await) | 96 | 93 | READONLY_TRUE_PROPERTY(target, "experimentalREPLAwait"); | 97 | 94 |
| |
Collapse file Copy file name to clipboardExpand all lines: src/node_options.cc +1-4Lines changed: 1 addition & 4 deletions | Original file line number | Diff line number | Diff line change |
|---|
@@ -109,10 +109,7 @@ EnvironmentOptionsParser::EnvironmentOptionsParser() { | 109 | 109 | "experimental ES Module support in vm module", | 110 | 110 | &EnvironmentOptions::experimental_vm_modules, | 111 | 111 | kAllowedInEnvironment); | 112 | | - AddOption("--experimental-worker", | 113 | | - "experimental threaded Worker support", | 114 | | - &EnvironmentOptions::experimental_worker, | 115 | | - kAllowedInEnvironment); | | 112 | + AddOption("--experimental-worker", "", NoOp{}, kAllowedInEnvironment); | 116 | 113 | AddOption("--expose-internals", "", &EnvironmentOptions::expose_internals); | 117 | 114 | AddOption("--http-parser", | 118 | 115 | "Select which HTTP parser to use; either 'legacy' or 'llhttp' " | |
Collapse file Copy file name to clipboardExpand all lines: src/node_options.h -1Lines changed: 0 additions & 1 deletion | Original file line number | Diff line number | Diff line change |
|---|
@@ -96,7 +96,6 @@ class EnvironmentOptions : public Options { | 96 | 96 | bool experimental_modules = false; | 97 | 97 | bool experimental_repl_await = false; | 98 | 98 | bool experimental_vm_modules = false; | 99 | | - bool experimental_worker = true; | 100 | 99 | bool expose_internals = false; | 101 | 100 | std::string http_parser = | 102 | 101 | #ifdef NODE_EXPERIMENTAL_HTTP_DEFAULT | |
|