fix: op_set_exit_code (#13034)

Fixes "op_set_exit_code" by sharing a single "Arc" between
all workers (via "op state") instead of having a "global" value stored in
"deno_runtime" crate. As a consequence setting an exit code is always
scoped to a tree of workers, instead of being overridable if there are
multiple worker tree (like in "deno test --jobs" subcommand).

Refactored "cli/main.rs" functions to return "Result<i32, AnyError>" instead
of "Result<(), AnyError>" so they can return exit code.

1 files changed, 0 insertions, 11 deletions

diff --git a/runtime/lib.rs b/runtime/lib.rs
index ad4a790da..58de3725b 100644
--- a/runtime/lib.rs
+++ b/runtime/lib.rs
@@ -1,7 +1,5 @@
 // Copyright 2018-2021 the Deno authors. All rights reserved. MIT license.
 
-use std::sync::atomic::AtomicI32;
-
 pub use deno_broadcast_channel;
 pub use deno_console;
 pub use deno_core;
@@ -32,12 +30,3 @@ pub mod worker;
 
 mod worker_bootstrap;
 pub use worker_bootstrap::BootstrapOptions;
-
-// The global may not be very elegant but:
-//
-// 1. op_exit() calls std::process::exit() so there is not much point storing
-//    the exit code in runtime state
-//
-// 2. storing it in runtime state makes retrieving it again in cli/main.rs
-//    unduly complicated
-pub static EXIT_CODE: AtomicI32 = AtomicI32::new(0);