From 51fcf2efdaae8bdd5270cfe64de1522eadd1775f 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 --- src/poem/read.rs | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) (limited to 'src/poem/read.rs') diff --git a/src/poem/read.rs b/src/poem/read.rs index 7f3ae32..99e9aa6 100644 --- a/src/poem/read.rs +++ b/src/poem/read.rs @@ -149,10 +149,8 @@ impl Appendable for Poem { lv.ep = verse.ep.clone(); lv.poems = verse.poems.clone(); lv.meter = verse.meter; - if verse.clause().is_some() { - for word in verse.clause().unwrap().iter() { - lv.stanza.push(word.to_string()); - } + for word in verse.clause().iter() { + lv.stanza.push(word.to_string()); } // Push verse(s) -- cgit v1.2.3