This is more efficient than draining and collecting the Vec. It also fixes the respective clippy lint.
Signed-off-by: Stefan Hanreich <s.hanre...@proxmox.com> Reviewed-by: Wolfgang Bumiller <w.bumil...@proxmox.com> --- proxmox-firewall/src/rule.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/proxmox-firewall/src/rule.rs b/proxmox-firewall/src/rule.rs index 3b947f0..b20a9c5 100644 --- a/proxmox-firewall/src/rule.rs +++ b/proxmox-firewall/src/rule.rs @@ -764,7 +764,7 @@ impl ToNftRules for FwMacro { fn to_nft_rules(&self, rules: &mut Vec<NftRule>, env: &NftRuleEnv) -> Result<(), Error> { log::trace!("applying macro: {self:?}"); - let initial_rules: Vec<NftRule> = rules.drain(..).collect(); + let initial_rules: Vec<NftRule> = std::mem::take(rules); for protocol in &self.code { let mut new_rules = initial_rules.to_vec(); -- 2.39.5 _______________________________________________ pve-devel mailing list pve-devel@lists.proxmox.com https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel