From 6610c0a164be386673a73044d3a751c25ccc5248 Mon Sep 17 00:00:00 2001
From: jdidion <github@didion.net>
Date: Thu, 7 Sep 2023 11:23:16 -0700
Subject: [PATCH] use inline array instead of `Vec` (Clippy error); get rid of
 unnecessary `mut`s (warning)

---
 src/lib/align/aligners/mod.rs                  |  4 ++--
 src/lib/align/aligners/multi_contig_aligner.rs | 10 +++++-----
 2 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/src/lib/align/aligners/mod.rs b/src/lib/align/aligners/mod.rs
index ed8630a..2346b5d 100644
--- a/src/lib/align/aligners/mod.rs
+++ b/src/lib/align/aligners/mod.rs
@@ -446,7 +446,7 @@ impl Aligners<'_, MatchParams> {
             let second_query_and_yend = (second_query, yend);
 
             // Align!
-            for (query, yend) in vec![first_query_and_yend, second_query_and_yend] {
+            for (query, yend) in [first_query_and_yend, second_query_and_yend] {
                 best_alignment = self
                     .realign_and_split_at_y(
                         &query,
@@ -487,7 +487,7 @@ impl Aligners<'_, MatchParams> {
             let second_query_and_ystart = (second_query, ystart);
 
             // Align!
-            for (query, ystart) in vec![first_query_and_ystart, second_query_and_ystart] {
+            for (query, ystart) in [first_query_and_ystart, second_query_and_ystart] {
                 best_alignment = self
                     .realign_and_split_at_y(
                         &query,
diff --git a/src/lib/align/aligners/multi_contig_aligner.rs b/src/lib/align/aligners/multi_contig_aligner.rs
index 0229fdb..c5ad73f 100644
--- a/src/lib/align/aligners/multi_contig_aligner.rs
+++ b/src/lib/align/aligners/multi_contig_aligner.rs
@@ -700,7 +700,7 @@ pub mod tests {
         // make these into test cases?
 
         // jump to the same contig and strand is prioritized
-        for mut contig in &mut aligner.contigs {
+        for contig in &mut aligner.contigs {
             contig.aligner.scoring = contig.aligner.scoring.set_jump_scores(-1, -2, -2);
         }
         let alignment = aligner.custom(&y1);
@@ -708,7 +708,7 @@ pub mod tests {
 
         // jump to the same contig and opposite strand is prioritized
         // starts in the middle of x2, then jumps back to the start of x1
-        for mut contig in &mut aligner.contigs {
+        for contig in &mut aligner.contigs {
             contig.aligner.scoring = contig.aligner.scoring.set_jump_scores(-2, -1, -2);
         }
         let alignment = aligner.custom(&y1);
@@ -716,14 +716,14 @@ pub mod tests {
 
         // jump to a different contig is prioritized
         // starts by aligning to x3 fully, then jumping to x1 and alinging to the last 5bp of x1
-        for mut contig in &mut aligner.contigs {
+        for contig in &mut aligner.contigs {
             contig.aligner.scoring = contig.aligner.scoring.set_jump_scores(-2, -2, -1);
         }
         let alignment = aligner.custom(&y1);
         assert_alignment(&alignment, 0, 15, 0, 10, 10 - 1, 2, "5=2c5J5=", 10);
 
         // jump to the same contig and strand is prioritized when the scores are the same
-        for mut contig in &mut aligner.contigs {
+        for contig in &mut aligner.contigs {
             contig.aligner.scoring = contig.aligner.scoring.set_jump_scores(-1, -1, -1);
         }
         let alignment = aligner.custom(&y1);
@@ -731,7 +731,7 @@ pub mod tests {
 
         // jump to the same contig and opposite is prioritized when the scores are the same
         // starts in the middle of x2, then jumps back to the start of x1
-        for mut contig in &mut aligner.contigs {
+        for contig in &mut aligner.contigs {
             contig.aligner.scoring = contig.aligner.scoring.set_jump_scores(-2, -1, -1);
         }
         let alignment = aligner.custom(&y1);