From 0b160cfdecaeb4b45623f1acccb910d092c32df8 Mon Sep 17 00:00:00 2001
From: Thomas Lamprecht <t.lamprecht@proxmox.com>
Date: Mon, 16 Jan 2023 11:05:25 +0100
Subject: [PATCH] tape inventory: code cleanup listing media assigned to a pool

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
---
 src/tape/inventory.rs | 20 ++++++++------------
 1 file changed, 8 insertions(+), 12 deletions(-)

diff --git a/src/tape/inventory.rs b/src/tape/inventory.rs
index 7f3e994a..e01a7f91 100644
--- a/src/tape/inventory.rs
+++ b/src/tape/inventory.rs
@@ -262,21 +262,17 @@ impl Inventory {
 
         for entry in self.map.values() {
             match entry.id.media_set_label {
-                None => continue, // not assigned to any pool
-                Some(ref set) => {
-                    if set.pool != pool {
-                        continue; // belong to another pool
-                    }
-
-                    if set.unassigned() {
-                        list.push(MediaId {
+                Some(ref set) if set.pool == pool => {
+                    let id = match set.unassigned() {
+                        true => MediaId {
                             label: entry.id.label.clone(),
                             media_set_label: None,
-                        })
-                    } else {
-                        list.push(entry.id.clone());
-                    }
+                        },
+                        false => entry.id.clone(),
+                    };
+                    list.push(id);
                 }
+                _ => continue, // not assigned to any pool or belongs to another pool
             }
         }