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 --- sq/src/commands/inspect.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'sq') diff --git a/sq/src/commands/inspect.rs b/sq/src/commands/inspect.rs index 7d136ab0..2d2037a9 100644 --- a/sq/src/commands/inspect.rs +++ b/sq/src/commands/inspect.rs @@ -46,7 +46,7 @@ pub fn inspect(m: &clap::ArgMatches, policy: &dyn Policy, output: &mut dyn io::W type_called = true; } let pp = openpgp::PacketPile::from( - ::std::mem::replace(&mut packets, Vec::new())); + std::mem::take(&mut packets)); let cert = openpgp::Cert::try_from(pp)?; inspect_cert(policy, output, &cert, print_certifications)?; -- cgit v1.2.3