diff --git a/src/cargo/core/source_id.rs b/src/cargo/core/source_id.rs index d03a0a5769c..19ec9ca051c 100644 --- a/src/cargo/core/source_id.rs +++ b/src/cargo/core/source_id.rs @@ -468,34 +468,33 @@ impl SourceId { /// Creates a new `SourceId` from this source with the given `precise`. pub fn with_git_precise(self, fragment: Option) -> SourceId { - SourceId::wrap(SourceIdInner { - precise: fragment.map(|f| Precise::GitUrlFragment(f)), - ..(*self.inner).clone() - }) + self.with_precise(&fragment.map(|f| Precise::GitUrlFragment(f))) } /// Creates a new `SourceId` from this source without a `precise`. pub fn without_precise(self) -> SourceId { - SourceId::wrap(SourceIdInner { - precise: None, - ..(*self.inner).clone() - }) + self.with_precise(&None) } /// Creates a new `SourceId` from this source without a `precise`. pub fn with_locked_precise(self) -> SourceId { - SourceId::wrap(SourceIdInner { - precise: Some(Precise::Locked), - ..(*self.inner).clone() - }) + self.with_precise(&Some(Precise::Locked)) } /// Creates a new `SourceId` from this source with the `precise` from some other `SourceId`. pub fn with_precise_from(self, v: Self) -> SourceId { - SourceId::wrap(SourceIdInner { - precise: v.inner.precise.clone(), - ..(*self.inner).clone() - }) + self.with_precise(&v.inner.precise) + } + + fn with_precise(self, precise: &Option) -> SourceId { + if &self.inner.precise == precise { + self + } else { + SourceId::wrap(SourceIdInner { + precise: precise.clone(), + ..(*self.inner).clone() + }) + } } /// When updating a lock file on a version using `cargo update --precise`