Update to rust 1.91.1

Author: fornwall

Cargo.toml

@@ -9,7 +9,7 @@ edition = "2024"
 homepage = "https://github.com/fornwall/advent-of-code"
 license = "MIT"
 repository = "https://github.com/fornwall/advent-of-code"
-rust-version = "1.89.0"
+rust-version = "1.91.1"
 version = "2024.21.0"
 
 [profile.release]

crates/core/src/common/character_recognition.rs

@@ -179,7 +179,7 @@ fn recognize_letter(
 }
 
 pub fn recognize(bytes: &[bool]) -> Result<String, String> {
-    if bytes.len() % (CHAR_WIDTH * CHAR_HEIGHT) != 0 {
+    if !bytes.len().is_multiple_of(CHAR_WIDTH * CHAR_HEIGHT) {
         return Err(format!(
             "Input length is not a multiple of {}",
             CHAR_WIDTH * CHAR_HEIGHT

crates/core/src/year2015/day23.rs

@@ -87,7 +87,7 @@ impl Computer {
                     instruction_pointer += offset;
                 }
                 Some(&Instruction::JumpIfEven(register, offset)) => {
-                    instruction_pointer += if self.registers[register as usize] % 2 == 0 {
+                    instruction_pointer += if self.registers[register as usize].is_multiple_of(2) {
                         offset
                     } else {
                         1

crates/core/src/year2018/day15.rs

@@ -127,7 +127,7 @@ impl Board {
     fn perform_round(&mut self) {
         self.round += 1;
         self.full_round = true;
-        let even_round = self.round % 2 == 0;
+        let even_round = self.round.is_multiple_of(2);
 
         for y in 0..self.height {
             for x in 0..self.width {

crates/core/src/year2019/day08.rs

@@ -6,7 +6,7 @@ const PIXELS_WIDE: usize = NUM_LETTERS * CHAR_WIDTH;
 const LAYER_SIZE: usize = PIXELS_WIDE * CHAR_HEIGHT;
 
 pub fn solve(input: &Input) -> Result<String, String> {
-    if input.text.len() % LAYER_SIZE != 0 {
+    if !input.text.len().is_multiple_of(LAYER_SIZE) {
         return Err(format!(
             "Invalid input - expected to be multiple of layer size ({LAYER_SIZE})"
         ));

crates/core/src/year2019/day17.rs

@@ -121,18 +121,18 @@ pub fn solve(input: &Input) -> Result<String, String> {
 
     let mut robot_direction = Direction::Up;
     let mut robot_position: (i32, i32) = (0, 0);
-    for y in 0..map.len() {
-        for x in 0..map[0].len() {
-            if map[y][x] == b'^' {
+    for (y, row) in map.iter().enumerate() {
+        for (x, &tile) in row.iter().enumerate() {
+            if tile == b'^' {
                 robot_direction = Direction::Up;
                 robot_position = (x as i32, y as i32);
-            } else if map[y][x] == b'v' {
+            } else if tile == b'v' {
                 robot_direction = Direction::Down;
                 robot_position = (x as i32, y as i32);
-            } else if map[y][x] == b'<' {
+            } else if tile == b'<' {
                 robot_direction = Direction::Left;
                 robot_position = (x as i32, y as i32);
-            } else if map[y][x] == b'>' {
+            } else if tile == b'>' {
                 robot_direction = Direction::Right;
                 robot_position = (x as i32, y as i32);
             }

crates/core/src/year2020/day20.rs

@@ -147,6 +147,7 @@ impl Tile {
                     && current.edges[2].matching.is_none() == (y + 1 == composed_image_width)
                     && current.edges[3].matching.is_none() == (x == 0)
                 {
+                    #[allow(clippy::useless_let_if_seq)]
                     let mut possible = true;
                     if x != 0
                         && let Some(tile_to_left) = composed_image.get(&(x - 1, y))

crates/core/src/year2022/day17.rs

@@ -57,7 +57,7 @@ pub fn solve(input: &Input) -> Result<u64, String> {
                             (current_rock_count - last_seen_rock_drop_iteration) as u64;
                         let remaining_rocks = target_rocks_count - current_rock_count as u64;
 
-                        if remaining_rocks % rocks_per_cycle == 0 {
+                        if remaining_rocks.is_multiple_of(rocks_per_cycle) {
                             let remaining_cycles = remaining_rocks / rocks_per_cycle;
                             let highest_rock_growth = grid.highest_rock - last_seen_highest_rock;
                             return Ok(grid.highest_rock as u64

crates/core/src/year2022/day19.rs

@@ -117,9 +117,8 @@ fn most_geodes_opened(blueprint: &Blueprint, minutes: u32) -> u32 {
             }
 
             let mut new_state = state;
-            for j in 0..4 {
-                new_state.ores[j] = new_state.ores[j]
-                    + state.robots[j] * (minutes_before_resources_obtained + 1)
+            for (j, ore) in new_state.ores.iter_mut().enumerate() {
+                *ore = *ore + state.robots[j] * (minutes_before_resources_obtained + 1)
                     - blueprint[robot_to_build_idx][j];
             }
             new_state.minutes_remaining -= minutes_before_resources_obtained + 1;

crates/core/src/year2022/day22.rs

@@ -229,6 +229,7 @@ impl Cube {
 
     fn step_forward(&mut self, steps: u8, mut direction: Direction) -> Direction {
         #![allow(clippy::panic)]
+        #![allow(clippy::useless_let_if_seq)]
         let mut delta = direction.delta();
         for _step in 0..steps {
             let mut new_position = (