From fd62379eafde6571f126df5650b80cfda9f74229 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Bartek=20Iwa=C5=84czuk?= Date: Thu, 14 Nov 2019 04:16:57 +0100 Subject: refactor: per-worker resource table (#3306) - removes global `RESOURCE_TABLE` - resource tables are now created per `Worker` in `State` - renames `CliResource` to `StreamResource` and moves all logic related to it to `cli/ops/io.rs` - removes `cli/resources.rs` - adds `state` argument to `op_read` and `op_write` and consequently adds `stateful_minimal_op` to `State` - IMPORTANT NOTE: workers don't have access to process stdio - this is caused by fact that dropping worker would close stdout for process (because it's constructed from raw handle, which closes underlying file descriptor on drop) --- cli/ops/resources.rs | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) (limited to 'cli/ops/resources.rs') diff --git a/cli/ops/resources.rs b/cli/ops/resources.rs index d92c6a83c..c35e9762c 100644 --- a/cli/ops/resources.rs +++ b/cli/ops/resources.rs @@ -1,7 +1,6 @@ // Copyright 2018-2019 the Deno authors. All rights reserved. MIT license. use super::dispatch_json::{JsonOp, Value}; use crate::ops::json_op; -use crate::resources::lock_resource_table; use crate::state::ThreadSafeState; use deno::*; @@ -10,11 +9,11 @@ pub fn init(i: &mut Isolate, s: &ThreadSafeState) { } fn op_resources( - _state: &ThreadSafeState, + state: &ThreadSafeState, _args: Value, _zero_copy: Option, ) -> Result { - let resource_table = lock_resource_table(); + let resource_table = state.lock_resource_table(); let serialized_resources = resource_table.entries(); Ok(JsonOp::Sync(json!(serialized_resources))) } -- cgit v1.2.3