Skip to content

Commit 04fbc5f

Browse files
committed
use make_mut for cleaner code
1 parent 736c3d7 commit 04fbc5f

File tree

1 file changed

+4
-8
lines changed

1 file changed

+4
-8
lines changed

src/cargo/core/resolver/context.rs

+4-8
Original file line numberDiff line numberDiff line change
@@ -68,9 +68,7 @@ impl Context {
6868
&*link
6969
);
7070
}
71-
let mut inner: Vec<_> = (**prev).clone();
72-
inner.push(summary.clone());
73-
*prev = Rc::new(inner);
71+
Rc::make_mut(prev).push(summary.clone());
7472
return Ok(false);
7573
}
7674
debug!("checking if {} is already activated", summary.package_id());
@@ -246,15 +244,13 @@ impl Context {
246244
if !reqs.used.is_empty() {
247245
let pkgid = s.package_id();
248246

249-
let set = self.resolve_features
247+
let set = Rc::make_mut(self.resolve_features
250248
.entry(pkgid.clone())
251-
.or_insert_with(|| Rc::new(HashSet::new()));
249+
.or_insert_with(|| Rc::new(HashSet::new())));
252250

253-
let mut inner: HashSet<_> = (**set).clone();
254251
for feature in reqs.used {
255-
inner.insert(InternedString::new(feature));
252+
set.insert(InternedString::new(feature));
256253
}
257-
*set = Rc::new(inner);
258254
}
259255

260256
Ok(ret)

0 commit comments

Comments
 (0)