From fb458537b458664dc8b43ea9de823fda66d663e6 Mon Sep 17 00:00:00 2001
From: Matthias Beyer <mail@beyermatthias.de>
Date: Sun, 7 Sep 2025 12:51:28 +0200
Subject: [PATCH] cli/patch: Replace manual building table from iterator

Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
---
 crates/radicle-cli/src/terminal/patch.rs | 24 +++++++++++++-----------
 1 file changed, 13 insertions(+), 11 deletions(-)

diff --git a/crates/radicle-cli/src/terminal/patch.rs b/crates/radicle-cli/src/terminal/patch.rs
index b4630cab..9bebb24f 100644
--- a/crates/radicle-cli/src/terminal/patch.rs
+++ b/crates/radicle-cli/src/terminal/patch.rs
@@ -306,18 +306,20 @@ pub fn get_update_message(
 
 /// List the given commits in a table.
 pub fn list_commits(commits: &[git::raw::Commit]) -> anyhow::Result<()> {
-    let mut table = term::Table::default();
+    commits
+        .iter()
+        .map(|commit| {
+            let message = commit
+                .summary_bytes()
+                .unwrap_or_else(|| commit.message_bytes());
 
-    for commit in commits {
-        let message = commit
-            .summary_bytes()
-            .unwrap_or_else(|| commit.message_bytes());
-        table.push([
-            term::format::secondary(term::format::oid(commit.id()).into()),
-            term::format::italic(String::from_utf8_lossy(message).to_string()),
-        ]);
-    }
-    table.print();
+            [
+                term::format::secondary(term::format::oid(commit.id()).into()),
+                term::format::italic(String::from_utf8_lossy(message).to_string()),
+            ]
+        })
+        .collect::<term::Table<2, _>>()
+        .print();
 
     Ok(())
 }