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 --- cli/tsc/99_main_compiler.js | 5 ----- 1 file changed, 5 deletions(-) (limited to 'cli/tsc') diff --git a/cli/tsc/99_main_compiler.js b/cli/tsc/99_main_compiler.js index ebb4d679d..f944b21b8 100644 --- a/cli/tsc/99_main_compiler.js +++ b/cli/tsc/99_main_compiler.js @@ -778,7 +778,6 @@ delete Object.prototype.__proto__; } hasStarted = true; languageService = ts.createLanguageService(host); - core.ops(); setLogDebug(debugFlag, "TSLS"); debug("serverInit()"); } @@ -793,13 +792,9 @@ delete Object.prototype.__proto__; throw new Error("The compiler runtime already started."); } hasStarted = true; - core.ops(); setLogDebug(!!debugFlag, "TS"); } - // Setup the compiler runtime during the build process. - core.ops(); - // A build time only op that provides some setup information that is used to // ensure the snapshot is setup properly. /** @type {{ buildSpecifier: string; libs: string[] }} */ -- cgit v1.2.3