Skip to content

Commit 36f0544

Browse files
committed
Inline trivial wrapper
1 parent 30c2900 commit 36f0544

File tree

1 file changed

+8
-19
lines changed

1 file changed

+8
-19
lines changed

src/config.rs

Lines changed: 8 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -753,28 +753,17 @@ impl<'a> Cfg<'a> {
753753
})
754754
}
755755

756-
async fn find_or_install_active_toolchain(
757-
&'a self,
758-
) -> Result<(Toolchain<'a>, ActiveReason)> {
759-
self.maybe_find_or_install_active_toolchain(&self.current_dir)
760-
.await?
761-
.ok_or_else(|| no_toolchain_error(self.process))
762-
}
763-
764756
#[cfg_attr(feature = "otel", tracing::instrument(skip_all))]
765-
async fn maybe_find_or_install_active_toolchain(
766-
&'a self,
767-
path: &Path,
768-
) -> Result<Option<(Toolchain<'a>, ActiveReason)>> {
769-
match self.find_override_config(path)? {
757+
async fn find_or_install_active_toolchain(&'a self) -> Result<(Toolchain<'a>, ActiveReason)> {
758+
match self.find_override_config(&self.current_dir)? {
770759
Some((override_config, reason)) => match override_config {
771760
OverrideCfg::PathBased(path_based_name) => {
772761
let toolchain = Toolchain::with_reason(self, path_based_name.into(), &reason)?;
773-
Ok(Some((toolchain, reason)))
762+
Ok((toolchain, reason))
774763
}
775764
OverrideCfg::Custom(custom_name) => {
776765
let toolchain = Toolchain::with_reason(self, custom_name.into(), &reason)?;
777-
Ok(Some((toolchain, reason)))
766+
Ok((toolchain, reason))
778767
}
779768
OverrideCfg::Official {
780769
toolchain,
@@ -785,22 +774,22 @@ impl<'a> Cfg<'a> {
785774
let toolchain = self
786775
.ensure_installed(toolchain, components, targets, profile)
787776
.await?;
788-
Ok(Some((toolchain, reason)))
777+
Ok((toolchain, reason))
789778
}
790779
},
791780
None => match self.get_default()? {
792-
None => Ok(None),
781+
None => Err(no_toolchain_error(self.process)),
793782
Some(ToolchainName::Custom(custom_name)) => {
794783
let reason = ActiveReason::Default;
795784
let toolchain = Toolchain::with_reason(self, custom_name.into(), &reason)?;
796-
Ok(Some((toolchain, reason)))
785+
Ok((toolchain, reason))
797786
}
798787
Some(ToolchainName::Official(toolchain_desc)) => {
799788
let reason = ActiveReason::Default;
800789
let toolchain = self
801790
.ensure_installed(toolchain_desc, vec![], vec![], None)
802791
.await?;
803-
Ok(Some((toolchain, reason)))
792+
Ok((toolchain, reason))
804793
}
805794
},
806795
}

0 commit comments

Comments
 (0)