actions/cargo-binstall
1
0
Fork 0
mirror of https://github.com/cargo-bins/cargo-binstall.git synced 2025-04-20 12:38:43 +00:00
Code Issues Projects Releases Packages Wiki Activity

Rm unnecessary Arc inside DesiredTargetsInner

Browse source 
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 <Jiahao_XU@outlook.com>
This commit is contained in:
Jiahao XU 2022-07-21 15:06:00 +10:00
parent 6a95bb07e0
commit eda7b9445a
No known key found for this signature in database
GPG key ID: 591C0B03040416D6
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/target.rs
View file

@ -11,7 +11,7 @@ pub const TARGET: &str = env!("TARGET");
#[derive(Debug, Clone)] #[derive(Debug, Clone)]
enum DesiredTargetsInner { enum DesiredTargetsInner {
AutoDetect(Arc<OnceCell<Vec<String>>>), AutoDetect(Arc<OnceCell<Vec<String>>>),
Initialized(Arc<Vec<String>>), Initialized(Vec<String>),
} }
#[derive(Debug, Clone)] #[derive(Debug, Clone)]
@ -19,7 +19,7 @@ pub struct DesiredTargets(DesiredTargetsInner);
impl DesiredTargets { impl DesiredTargets {
fn initialized(targets: Vec<String>) -> Self { fn initialized(targets: Vec<String>) -> Self {
Self(DesiredTargetsInner::Initialized(Arc::new(targets))) Self(DesiredTargetsInner::Initialized(targets))
} }
fn auto_detect() -> Self { fn auto_detect() -> Self {