diff options
author | Christian Dürr <102963075+cd-work@users.noreply.github.com> | 2022-10-15 21:19:03 +0000 |
---|---|---|
committer | GitHub <noreply@github.com> | 2022-10-15 23:19:03 +0200 |
commit | 6d2656fd56e8ac0f1b1443e28d474bf3ceca89bb (patch) | |
tree | 394b01a0fcf33fb684c7048528d02a2f9b3dd0cb /runtime/worker.rs | |
parent | 872dc9b1df1dec5466970f085875e50b9436e967 (diff) |
refactor: Add default implementation for WorkerOptions (#14860)
This adds an implementation of `Default` for both `WorkerOptions` and
`BootstrapOptions`. Since both of these structs are rather big, this
should make it easier for people unfamiliar with the internals to focus
on the options relevant to them.
As a user of `deno_runtime` I feel like these should serve as good
defaults, getting people them started without having to tweak the
runtime. Additionally even if some changes are made, the usage of
`..Default::default()` will significantly help with code clarity and
verbosity.
Diffstat (limited to 'runtime/worker.rs')
-rw-r--r-- | runtime/worker.rs | 38 |
1 files changed, 37 insertions, 1 deletions
diff --git a/runtime/worker.rs b/runtime/worker.rs index 9b2755939..36833da32 100644 --- a/runtime/worker.rs +++ b/runtime/worker.rs @@ -18,6 +18,7 @@ use deno_core::serde_v8; use deno_core::v8; use deno_core::CompiledWasmModuleStore; use deno_core::Extension; +use deno_core::FsModuleLoader; use deno_core::GetErrorClassFn; use deno_core::JsRuntime; use deno_core::LocalInspectorSession; @@ -105,6 +106,41 @@ fn grab_cb( v8::Global::new(scope, cb) } +impl Default for WorkerOptions { + fn default() -> Self { + Self { + web_worker_preload_module_cb: Arc::new(|_| { + unimplemented!("web workers are not supported") + }), + web_worker_pre_execute_module_cb: Arc::new(|_| { + unimplemented!("web workers are not supported") + }), + create_web_worker_cb: Arc::new(|_| { + unimplemented!("web workers are not supported") + }), + module_loader: Rc::new(FsModuleLoader), + seed: None, + unsafely_ignore_certificate_errors: Default::default(), + should_break_on_first_statement: Default::default(), + compiled_wasm_module_store: Default::default(), + shared_array_buffer_store: Default::default(), + maybe_inspector_server: Default::default(), + format_js_error_fn: Default::default(), + get_error_class_fn: Default::default(), + origin_storage_dir: Default::default(), + cache_storage_dir: Default::default(), + broadcast_channel: Default::default(), + source_map_getter: Default::default(), + root_cert_store: Default::default(), + npm_resolver: Default::default(), + blob_store: Default::default(), + extensions: Default::default(), + bootstrap: Default::default(), + stdio: Default::default(), + } + } +} + impl MainWorker { pub fn bootstrap_from_options( main_module: ModuleSpecifier, @@ -533,7 +569,7 @@ mod tests { create_web_worker_cb: Arc::new(|_| unreachable!()), maybe_inspector_server: None, should_break_on_first_statement: false, - module_loader: Rc::new(deno_core::FsModuleLoader), + module_loader: Rc::new(FsModuleLoader), npm_resolver: None, get_error_class_fn: None, cache_storage_dir: None, |