From d4e9d5470eeebc02962a015182473364828d6d7d Mon Sep 17 00:00:00 2001 From: Dominik Csapak Date: Thu, 28 Oct 2021 11:47:52 +0200 Subject: [PATCH] PruneMark: use copied values instead of references the type is small enough Signed-off-by: Dominik Csapak Signed-off-by: Wolfgang Bumiller --- pbs-datastore/src/prune.rs | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/pbs-datastore/src/prune.rs b/pbs-datastore/src/prune.rs index e66ed408..c144b3db 100644 --- a/pbs-datastore/src/prune.rs +++ b/pbs-datastore/src/prune.rs @@ -11,12 +11,12 @@ use super::BackupInfo; pub enum PruneMark { Protected, Keep, KeepPartial, Remove } impl PruneMark { - pub fn keep(&self) -> bool { - *self != PruneMark::Remove + pub fn keep(self) -> bool { + self != PruneMark::Remove } - pub fn protected(&self) -> bool { - *self == PruneMark::Protected + pub fn protected(self) -> bool { + self == PruneMark::Protected } } @@ -202,7 +202,7 @@ pub fn compute_prune_info( let mark = if info.protected { PruneMark::Protected } else { - *mark.get(&backup_id).unwrap_or(&PruneMark::Remove) + mark.get(&backup_id).copied().unwrap_or(PruneMark::Remove) }; (info, mark)