From 83ab2d76e4573271de533a096f080e8a4c0f47d2 Mon Sep 17 00:00:00 2001
From: Abdul-Rahman Sibahi <asibahi@users.noreply.github.com>
Date: Mon, 2 Dec 2024 18:56:23 +0300
Subject: [PATCH 1/2] Update runner.rs

Use `Copy` bound instead of `Clone`. As references are `Copy`.
---
 src/template/runner.rs | 10 +++-------
 1 file changed, 3 insertions(+), 7 deletions(-)

diff --git a/src/template/runner.rs b/src/template/runner.rs
index 0a48767..4371d90 100644
--- a/src/template/runner.rs
+++ b/src/template/runner.rs
@@ -9,7 +9,7 @@ use std::{cmp, env, process};
 use crate::template::ANSI_BOLD;
 use crate::template::{aoc_cli, Day, ANSI_ITALIC, ANSI_RESET};
 
-pub fn run_part<I: Clone, T: Display>(func: impl Fn(I) -> Option<T>, input: I, day: Day, part: u8) {
+pub fn run_part<I: Copy, T: Display>(func: impl Fn(I) -> Option<T>, input: I, day: Day, part: u8) {
     let part_str = format!("Part {part}");
 
     let (result, duration, samples) =
@@ -25,15 +25,13 @@ pub fn run_part<I: Clone, T: Display>(func: impl Fn(I) -> Option<T>, input: I, d
 /// Run a solution part. The behavior differs depending on whether we are running a release or debug build:
 ///  1. in debug, the function is executed once.
 ///  2. in release, the function is benched (approx. 1 second of execution time or 10 samples, whatever take longer.)
-fn run_timed<I: Clone, T>(
+fn run_timed<I: Copy, T>(
     func: impl Fn(I) -> T,
     input: I,
     hook: impl Fn(&T),
 ) -> (T, Duration, u128) {
     let timer = Instant::now();
     let result = {
-        let input = input.clone();
-
         #[cfg(feature = "dhat-heap")]
         let _profiler = dhat::Profiler::new_heap();
 
@@ -52,7 +50,7 @@ fn run_timed<I: Clone, T>(
     (result, run.0, run.1)
 }
 
-fn bench<I: Clone, T>(func: impl Fn(I) -> T, input: I, base_time: &Duration) -> (Duration, u128) {
+fn bench<I: Copy, T>(func: impl Fn(I) -> T, input: I, base_time: &Duration) -> (Duration, u128) {
     let mut stdout = stdout();
 
     print!(" > {ANSI_ITALIC}benching{ANSI_RESET}");
@@ -64,8 +62,6 @@ fn bench<I: Clone, T>(func: impl Fn(I) -> T, input: I, base_time: &Duration) ->
     let mut timers: Vec<Duration> = vec![];
 
     for _ in 0..bench_iterations {
-        // need a clone here to make the borrow checker happy.
-        let cloned = input.clone();
         let timer = Instant::now();
         black_box(func(black_box(cloned)));
         timers.push(timer.elapsed());

From 16f80de467facc2040ab9a426e85a80e3b6d724e Mon Sep 17 00:00:00 2001
From: Abdul-Rahman Sibahi <asibahi@users.noreply.github.com>
Date: Tue, 3 Dec 2024 00:29:23 +0300
Subject: [PATCH 2/2] Update src/template/runner.rs

Co-authored-by: Matt Clarke <mattjclarke94@gmail.com>
---
 src/template/runner.rs | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/template/runner.rs b/src/template/runner.rs
index 4371d90..43a2bb4 100644
--- a/src/template/runner.rs
+++ b/src/template/runner.rs
@@ -63,7 +63,7 @@ fn bench<I: Copy, T>(func: impl Fn(I) -> T, input: I, base_time: &Duration) -> (
 
     for _ in 0..bench_iterations {
         let timer = Instant::now();
-        black_box(func(black_box(cloned)));
+        black_box(func(black_box(input)));
         timers.push(timer.elapsed());
     }