Use split_last

apache · Sep 23, 2024 · 0a45845 · 0a45845
1 parent e8eeb9a
commit 0a45845
Showing 1 changed file with 7 additions and 5 deletions.
diff --git a/crates/iceberg/src/catalog/mod.rs b/crates/iceberg/src/catalog/mod.rs
@@ -137,11 +137,13 @@ impl NamespaceIdent {
     /// Get the parent of this namespace.
     /// Returns None if this namespace only has a single element and thus has no parent.
     pub fn parent(&self) -> Option<Self> {
-        if self.0.len() <= 1 {
-            None
-        } else {
-            Some(Self(self.0[..self.0.len() - 1].to_vec()))
-        }
+        self.0.split_last().and_then(|(_, parent)| {
+            if parent.is_empty() {
+                None
+            } else {
+                Some(Self(parent.to_vec()))
+            }
+        })
     }
 }