From 83bece56b01f6997cb71e9289a4d83a398cde0c8 Mon Sep 17 00:00:00 2001 From: Aaron O'Mullan Date: Sun, 25 Apr 2021 22:00:05 +0200 Subject: refactor(core): move op cache sync responsibility to rust space (#10340) Even if bootstrapping the JS runtime is low level, it's an abstraction leak of core to require users to call `Deno.core.ops()` in JS space. So instead we're introducing a `JsRuntime::sync_ops_cache()` method, once we have runtime extensions a new runtime will ensure the ops cache is setup (for the provided extensions) and then loading/unloading plugins should be the only operations that require op cache syncs --- bench_util/src/js_runtime.rs | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) (limited to 'bench_util') diff --git a/bench_util/src/js_runtime.rs b/bench_util/src/js_runtime.rs index 415f4a135..d5509d624 100644 --- a/bench_util/src/js_runtime.rs +++ b/bench_util/src/js_runtime.rs @@ -16,13 +16,7 @@ pub fn create_js_runtime(setup: impl FnOnce(&mut JsRuntime)) -> JsRuntime { setup(&mut rt); // Init ops - rt.execute( - "init", - r#" - Deno.core.ops(); - "#, - ) - .unwrap(); + rt.sync_ops_cache(); rt } -- cgit v1.2.3