From 97ca3fa5197599ec28b19d5b2048f7c24e4c8bbc Mon Sep 17 00:00:00 2001
From: Rory Dudley
Date: Mon, 2 Sep 2024 22:23:01 -0600
Subject: Refactor Verse::clause()

There is no reason to return an option for clause(), since it makes us
perform a match twice, and since a blank vector is perfectly acceptable
in all cases where the verse at hand may not have a clause.

Signed-off-by: Rory Dudley <rory@netc.lu>
---
 src/poem/elements/verse.rs       | 8 ++++----
 src/poem/elements/verse/logic.rs | 2 +-
 2 files changed, 5 insertions(+), 5 deletions(-)

(limited to 'src/poem/elements')

diff --git a/src/poem/elements/verse.rs b/src/poem/elements/verse.rs
index 1c5bcc1..9ebd31d 100644
--- a/src/poem/elements/verse.rs
+++ b/src/poem/elements/verse.rs
@@ -140,11 +140,11 @@ impl Verse {
     /// Get the [Verse]'s clause
     ///
     /// Return program arguments, if they exist
-    pub fn clause(&self) -> Option<Vec<String>> {
+    pub fn clause(&self) -> Vec<String> {
         match self.stanza.len() {
-            0 => None,
-            1 => None,
-            _ => Some(self.stanza[1..].to_vec()),
+            0 => vec![],
+            1 => vec![],
+            _ => self.stanza[1..].to_vec(),
         }
     }
 
diff --git a/src/poem/elements/verse/logic.rs b/src/poem/elements/verse/logic.rs
index e7ed4df..41b736b 100644
--- a/src/poem/elements/verse/logic.rs
+++ b/src/poem/elements/verse/logic.rs
@@ -15,7 +15,7 @@
 #[macro_export]
 macro_rules! incant {
     ($verb:expr, $command:expr, $out:expr, $pids:expr, $env:expr, $self:expr) => {{
-        $command.args($self.clause().unwrap_or(vec![]));
+        $command.args($self.clause());
 
         // Determine couplet status
         if $self.couplet == 1 {
-- 
cgit v1.2.3