summaryrefslogtreecommitdiff
path: root/cli/main.rs
diff options
context:
space:
mode:
authorDavid Sherret <dsherret@users.noreply.github.com>2023-04-14 16:22:33 -0400
committerGitHub <noreply@github.com>2023-04-14 16:22:33 -0400
commit136dce67cec749dce5989ea29e88359ef79a0045 (patch)
tree38e96bbbf22dc06cdba418a35467b215f1335549 /cli/main.rs
parenta4111442191fff300132259752e6d2d5613d1871 (diff)
refactor: break up `ProcState` (#18707)
1. Breaks up functionality within `ProcState` into several other structs to break out the responsibilities (`ProcState` is only a data struct now). 2. Moves towards being able to inject dependencies more easily and have functionality only require what it needs. 3. Exposes `Arc<T>` around the "service structs" instead of it being embedded within them. The idea behind embedding them was to reduce the verbosity of needing to pass around `Arc<...>`, but I don't think it was exactly working and as we move more of these structs to be more injectable I don't think the extra verbosity will be a big deal.
Diffstat (limited to 'cli/main.rs')
-rw-r--r--cli/main.rs10
1 files changed, 7 insertions, 3 deletions
diff --git a/cli/main.rs b/cli/main.rs
index 60eb9288f..5e088d891 100644
--- a/cli/main.rs
+++ b/cli/main.rs
@@ -70,13 +70,17 @@ async fn run_subcommand(flags: Flags) -> Result<i32, AnyError> {
}
DenoSubcommand::Cache(cache_flags) => {
let ps = ProcState::from_flags(flags).await?;
- ps.load_and_type_check_files(&cache_flags.files).await?;
- ps.emitter.cache_module_emits(&ps.graph())?;
+ ps.module_load_preparer
+ .load_and_type_check_files(&cache_flags.files)
+ .await?;
+ ps.emitter.cache_module_emits(&ps.graph_container.graph())?;
Ok(0)
}
DenoSubcommand::Check(check_flags) => {
let ps = ProcState::from_flags(flags).await?;
- ps.load_and_type_check_files(&check_flags.files).await?;
+ ps.module_load_preparer
+ .load_and_type_check_files(&check_flags.files)
+ .await?;
Ok(0)
}
DenoSubcommand::Compile(compile_flags) => {