From eda7b9445a1c7d7a1797574f7696c39af1dba63b Mon Sep 17 00:00:00 2001 From: Jiahao XU Date: Thu, 21 Jul 2022 15:06:00 +1000 Subject: [PATCH] Rm unnecessary `Arc` inside` DesiredTargetsInner` Since `DesiredTargets` is now stored in `binstall::Options`, which itself is wrapped in an `Arc`, `DesiredTargetsInner::Initialized` no longer needs an `Arc` for O(1) `clone`. Signed-off-by: Jiahao XU --- src/target.rs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/target.rs b/src/target.rs index c6bf8dd9..fdca606f 100644 --- a/src/target.rs +++ b/src/target.rs @@ -11,7 +11,7 @@ pub const TARGET: &str = env!("TARGET"); #[derive(Debug, Clone)] enum DesiredTargetsInner { AutoDetect(Arc>>), - Initialized(Arc>), + Initialized(Vec), } #[derive(Debug, Clone)] @@ -19,7 +19,7 @@ pub struct DesiredTargets(DesiredTargetsInner); impl DesiredTargets { fn initialized(targets: Vec) -> Self { - Self(DesiredTargetsInner::Initialized(Arc::new(targets))) + Self(DesiredTargetsInner::Initialized(targets)) } fn auto_detect() -> Self {