From d11e89127d723d84afe6e5603b731c014a54f9da Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Bartek=20Iwa=C5=84czuk?= Date: Sat, 18 Mar 2023 10:59:51 -0400 Subject: Revert "perf(core): preserve ops between snapshots (#18080)" (#18267) This reverts commit 4b6305f4f25fc76f974bbdcc9cdb139d5ab8f5f4. --- cli/build.rs | 3 ++- cli/lsp/tsc.rs | 3 --- cli/ops/bench.rs | 3 --- cli/ops/mod.rs | 3 --- cli/ops/testing.rs | 3 --- cli/tsc/mod.rs | 11 +---------- 6 files changed, 3 insertions(+), 23 deletions(-) (limited to 'cli') diff --git a/cli/build.rs b/cli/build.rs index 173c8b85d..c2269aca3 100644 --- a/cli/build.rs +++ b/cli/build.rs @@ -5,6 +5,7 @@ use std::path::PathBuf; use deno_core::snapshot_util::*; use deno_core::Extension; +use deno_core::ExtensionBuilder; use deno_core::ExtensionFileSource; use deno_core::ExtensionFileSourceCode; use deno_runtime::deno_cache::SqliteBackedCache; @@ -317,7 +318,7 @@ deno_core::extension!( dir "js", "40_testing.js" ], - customizer = |ext: &mut deno_core::ExtensionBuilder| { + customizer = |ext: &mut ExtensionBuilder| { ext.esm(vec![ExtensionFileSource { specifier: "ext:cli/runtime/js/99_main.js", code: ExtensionFileSourceCode::LoadedFromFsDuringSnapshot( diff --git a/cli/lsp/tsc.rs b/cli/lsp/tsc.rs index c8d8103b9..91eb6e24b 100644 --- a/cli/lsp/tsc.rs +++ b/cli/lsp/tsc.rs @@ -2843,9 +2843,6 @@ deno_core::extension!(deno_tsc, options.performance, )); }, - customizer = |ext: &mut deno_core::ExtensionBuilder| { - ext.force_op_registration(); - }, ); /// Instruct a language server runtime to start the language server and provide diff --git a/cli/ops/bench.rs b/cli/ops/bench.rs index 86498cd7c..6fa9edee8 100644 --- a/cli/ops/bench.rs +++ b/cli/ops/bench.rs @@ -38,9 +38,6 @@ deno_core::extension!(deno_bench, state.put(options.sender); state.put(options.filter); }, - customizer = |ext: &mut deno_core::ExtensionBuilder| { - ext.force_op_registration(); - }, ); #[derive(Clone)] diff --git a/cli/ops/mod.rs b/cli/ops/mod.rs index 4d5595e09..c12409514 100644 --- a/cli/ops/mod.rs +++ b/cli/ops/mod.rs @@ -21,9 +21,6 @@ deno_core::extension!(deno_cli, state = |state, options| { state.put(options.ps); }, - customizer = |ext: &mut deno_core::ExtensionBuilder| { - ext.force_op_registration(); - }, ); #[op] diff --git a/cli/ops/testing.rs b/cli/ops/testing.rs index 8b5c95fea..0849f1c7a 100644 --- a/cli/ops/testing.rs +++ b/cli/ops/testing.rs @@ -44,9 +44,6 @@ deno_core::extension!(deno_test, state.put(options.fail_fast_tracker); state.put(options.filter); }, - customizer = |ext: &mut deno_core::ExtensionBuilder| { - ext.force_op_registration(); - }, ); #[derive(Clone)] diff --git a/cli/tsc/mod.rs b/cli/tsc/mod.rs index 48ab131fe..45589780f 100644 --- a/cli/tsc/mod.rs +++ b/cli/tsc/mod.rs @@ -114,13 +114,7 @@ pub fn get_types_declaration_file_text(unstable: bool) -> String { } fn get_asset_texts_from_new_runtime() -> Result, AnyError> { - deno_core::extension!( - deno_cli_tsc, - ops_fn = deno_ops, - customizer = |ext: &mut deno_core::ExtensionBuilder| { - ext.force_op_registration(); - }, - ); + deno_core::extension!(deno_cli_tsc, ops_fn = deno_ops,); // the assets are stored within the typescript isolate, so take them out of there let mut runtime = JsRuntime::new(RuntimeOptions { @@ -852,9 +846,6 @@ pub fn exec(request: Request) -> Result { .unwrap(), )); }, - customizer = |ext: &mut deno_core::ExtensionBuilder| { - ext.force_op_registration(); - }, ); let startup_source = "globalThis.startup({ legacyFlag: false })"; -- cgit v1.2.3