From 58946b2da1bf12c42854c03e67d6ba0a540ce317 Mon Sep 17 00:00:00 2001 From: Nora Widdecke Date: Mon, 22 Nov 2021 20:41:01 +0100 Subject: Use std::mem::take instead of std::mem::replace, for clarity. - Replace let bar = std::mem::replace(&foo, Default::Default()); with let bar = std::mem::take(&foo); The new version seems a little clearer. - Found by clippy: https://rust-lang.github.io/rust-clippy/master/index.html#mem_replace_with_default --- openpgp/src/parse/hashed_reader.rs | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) (limited to 'openpgp/src/parse') diff --git a/openpgp/src/parse/hashed_reader.rs b/openpgp/src/parse/hashed_reader.rs index 0568829d..00f56427 100644 --- a/openpgp/src/parse/hashed_reader.rs +++ b/openpgp/src/parse/hashed_reader.rs @@ -427,8 +427,7 @@ mod test { let cookie = reader.cookie_mut(); - let mut hashes = mem::replace(&mut cookie.sig_group_mut().hashes, - Default::default()); + let mut hashes = std::mem::take(&mut cookie.sig_group_mut().hashes); for mode in hashes.iter_mut() { let hash = mode.as_mut(); let algo = hash.algo(); -- cgit v1.2.3