From 9b4f09cf736e68bdbd246dca17d7a3c6b8eba3ea Mon Sep 17 00:00:00 2001
From: Benedikt Peetz <benedikt.peetz@b-peetz.de>
Date: Tue, 24 Jun 2025 14:50:07 +0200
Subject: fix(yt_dlp/post_processors/dearrow): Don't try to access the drained
 vec
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

If all titles were declared “bad” the pp previously tried to access the
first title in the array, which was already drained. We now simply clone
the array to facilitate this usage.
---
 crates/yt_dlp/src/post_processors/dearrow.rs | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

(limited to 'crates')

diff --git a/crates/yt_dlp/src/post_processors/dearrow.rs b/crates/yt_dlp/src/post_processors/dearrow.rs
index ab5478b..3cac745 100644
--- a/crates/yt_dlp/src/post_processors/dearrow.rs
+++ b/crates/yt_dlp/src/post_processors/dearrow.rs
@@ -59,8 +59,9 @@ pub fn unwrapped_process(info: PyRef<PyDict>, vm: &VirtualMachine) -> Result<PyR
     output.titles.reverse();
 
     let title_len = output.titles.len();
+    let mut iterator = output.titles.clone();
     let selected = loop {
-        let Some(title) = output.titles.pop() else {
+        let Some(title) = iterator.pop() else {
             break false;
         };
 
@@ -145,7 +146,7 @@ struct CasualVote {
     title: String,
 }
 
-#[derive(Serialize, Deserialize)]
+#[derive(Serialize, Deserialize, Clone)]
 struct Title {
     /// Note: Titles will sometimes contain > before a word.
     /// This tells the auto-formatter to not format a word.
-- 
cgit 1.4.1