diff options
author | David Sherret <dsherret@users.noreply.github.com> | 2023-04-14 16:22:33 -0400 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-04-14 16:22:33 -0400 |
commit | 136dce67cec749dce5989ea29e88359ef79a0045 (patch) | |
tree | 38e96bbbf22dc06cdba418a35467b215f1335549 /cli/main.rs | |
parent | a4111442191fff300132259752e6d2d5613d1871 (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.rs | 10 |
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) => { |