From 3c1c76d41ce0b828da651831af9aa0359014484b Mon Sep 17 00:00:00 2001 From: Chris Cochrun Date: Mon, 20 Oct 2025 15:02:00 -0500 Subject: [PATCH] presentations don't need to create the slide at all if not in range --- src/core/presentations.rs | 17 +++++++++-------- 1 file changed, 9 insertions(+), 8 deletions(-) diff --git a/src/core/presentations.rs b/src/core/presentations.rs index b3c4a35..09030fb 100644 --- a/src/core/presentations.rs +++ b/src/core/presentations.rs @@ -204,7 +204,14 @@ impl ServiceTrait for Presentation { let pages = document.pages().into_diagnostic()?; debug!(?pages); let pages: Vec = pages - .filter_map(|page| { + .enumerate() + .filter_map(|(index, page)| { + if (index as i32) < starting_index { + return None; + } else if (index as i32) > ending_index { + return None; + }; + let Some(page) = page.ok() else { return None; }; @@ -227,13 +234,7 @@ impl ServiceTrait for Presentation { .collect(); let mut slides: Vec = vec![]; - for (index, page) in pages.into_iter().enumerate() { - if (index as i32) < starting_index { - continue; - } else if (index as i32) > ending_index { - continue; - }; - + for (index, page) in pages.into_iter() { let slide = SlideBuilder::new() .background( Background::try_from(self.path.clone())