From a556388635b1f7543a361985d1ef3e2c58ddd386 Mon Sep 17 00:00:00 2001
From: Owen Jacobson <owen@grimoire.ca>
Date: Wed, 24 Jul 2024 19:30:20 -0400
Subject: Stop converting a vec into a vec needlessly during dry-run

---
 src/apply.rs | 13 ++++++-------
 1 file changed, 6 insertions(+), 7 deletions(-)

(limited to 'src/apply.rs')

diff --git a/src/apply.rs b/src/apply.rs
index 85c34d7..48cc1bf 100644
--- a/src/apply.rs
+++ b/src/apply.rs
@@ -1,3 +1,5 @@
+use std::fmt::Debug;
+
 use anyhow::Result;
 use aws_sdk_route53::types::{Change, ChangeAction, ChangeBatch, ResourceRecordSet};
 // Needed until try_collect is stable, see <https://github.com/rust-lang/rust/issues/94047>
@@ -20,8 +22,8 @@ impl ApplyMode {
     ) -> Result<()>
     where
         C: Route53,
-        R: IntoIterator<Item = ResourceRecordSet>,
-        I: IntoIterator<Item = ResourceRecordSet>,
+        R: IntoIterator<Item = ResourceRecordSet> + Debug,
+        I: IntoIterator<Item = ResourceRecordSet> + Debug,
     {
         match self {
             ApplyMode::DryRun => dry_run(zone_id, remove_records, insert_records).await,
@@ -32,12 +34,9 @@ impl ApplyMode {
 
 async fn dry_run<R, I>(zone_id: &str, remove_records: R, insert_records: I) -> Result<()>
 where
-    R: IntoIterator<Item = ResourceRecordSet>,
-    I: IntoIterator<Item = ResourceRecordSet>,
+    R: IntoIterator<Item = ResourceRecordSet> + Debug,
+    I: IntoIterator<Item = ResourceRecordSet> + Debug,
 {
-    let remove_records: Vec<_> = remove_records.into_iter().collect();
-    let insert_records: Vec<_> = insert_records.into_iter().collect();
-
     println!("ZONE: {}", zone_id);
     println!("REMOVE: {:#?}", remove_records);
     println!("INSERT: {:#?}", insert_records);
-- 
cgit v1.2.3