all lists on lists.proxmox.com
 help / color / mirror / Atom feed
* [pbs-devel] [PATCH proxmox-backup 1/4] PruneMark: use copied values instead of references
@ 2021-10-28  9:47 Dominik Csapak
  2021-10-28  9:47 ` [pbs-devel] [PATCH proxmox-backup 2/4] PruneMark: implement display without the write! macro Dominik Csapak
                   ` (3 more replies)
  0 siblings, 4 replies; 5+ messages in thread
From: Dominik Csapak @ 2021-10-28  9:47 UTC (permalink / raw)
  To: pbs-devel

the type is small enough

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
---
 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)
-- 
2.30.2





^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2021-10-28 10:57 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-10-28  9:47 [pbs-devel] [PATCH proxmox-backup 1/4] PruneMark: use copied values instead of references Dominik Csapak
2021-10-28  9:47 ` [pbs-devel] [PATCH proxmox-backup 2/4] PruneMark: implement display without the write! macro Dominik Csapak
2021-10-28  9:47 ` [pbs-devel] [PATCH proxmox-backup 3/4] backup-client: use () instead of Value as return type Dominik Csapak
2021-10-28  9:47 ` [pbs-devel] [PATCH proxmox-backup 4/4] api: admin/datastore: reuse 'is_protected' implementation Dominik Csapak
2021-10-28 10:57 ` [pbs-devel] applied-series: [PATCH proxmox-backup 1/4] PruneMark: use copied values instead of references Wolfgang Bumiller

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal