From bb356427dfb33b545b51452b10dccae0b2b0dbe3 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Max=20K=C3=A4nner?= Date: Sat, 5 Oct 2024 20:19:49 +0200 Subject: [PATCH] cargo clippy --fix --- battlesnake/src/logic.rs | 2 +- battlesnake/src/main.rs | 2 +- battlesnake/src/simulation.rs | 18 +++++++++--------- 3 files changed, 11 insertions(+), 11 deletions(-) diff --git a/battlesnake/src/logic.rs b/battlesnake/src/logic.rs index de24baa..0967765 100644 --- a/battlesnake/src/logic.rs +++ b/battlesnake/src/logic.rs @@ -30,7 +30,7 @@ use crate::{ // info is called when you create your Battlesnake on play.battlesnake.com // and controls your Battlesnake's appearance // TIP: If you open your Battlesnake URL in a browser you should see this data -pub fn info() -> Value { +#[must_use] pub fn info() -> Value { info!("INFO"); json!({ diff --git a/battlesnake/src/main.rs b/battlesnake/src/main.rs index d4e5440..c8b5868 100644 --- a/battlesnake/src/main.rs +++ b/battlesnake/src/main.rs @@ -7,7 +7,7 @@ use rocket::{ get, http::Status, launch, post, routes, - serde::{json::Json, Deserialize}, + serde::json::Json, tokio::task, }; use serde_json::Value; diff --git a/battlesnake/src/simulation.rs b/battlesnake/src/simulation.rs index f89976e..164a22a 100644 --- a/battlesnake/src/simulation.rs +++ b/battlesnake/src/simulation.rs @@ -14,7 +14,7 @@ pub struct SnakeToken { } impl SnakeToken { - pub fn from_board(board: &crate::Board) -> BTreeMap { + #[must_use] pub fn from_board(board: &crate::Board) -> BTreeMap { board .snakes .iter() @@ -52,7 +52,7 @@ pub struct Board { } impl Board { - pub fn from_game_board( + #[must_use] pub fn from_game_board( board: &crate::Board, token_map: &BTreeMap, turn: i32, @@ -86,16 +86,16 @@ impl Board { } } - pub const fn turn(&self) -> i32 { + #[must_use] pub const fn turn(&self) -> i32 { self.turn } #[allow(clippy::cast_sign_loss)] - pub const fn spaces(&self) -> usize { + #[must_use] pub const fn spaces(&self) -> usize { self.height as usize * self.width as usize } - pub fn alive_snakes(&self) -> usize { + #[must_use] pub fn alive_snakes(&self) -> usize { self.snakes.len() } @@ -103,7 +103,7 @@ impl Board { self.snakes.keys().copied() } - pub fn snake_length(&self, token: SnakeToken) -> Option { + #[must_use] pub fn snake_length(&self, token: SnakeToken) -> Option { self.snakes.get(&token).map(|snake| snake.body.len()) } @@ -196,7 +196,7 @@ impl Board { } } - pub fn possible_actions(&self) -> BTreeMap> { + #[must_use] pub fn possible_actions(&self) -> BTreeMap> { let mut actions: BTreeMap<_, BTreeSet<_>> = self .snakes .keys() @@ -272,7 +272,7 @@ pub struct Battlesnake { } impl Battlesnake { - pub fn from_game_snake(snake: &crate::Battlesnake) -> Self { + #[must_use] pub fn from_game_snake(snake: &crate::Battlesnake) -> Self { let body: VecDeque<_> = snake.body.iter().copied().collect(); debug_assert_eq!(body.len(), usize::try_from(snake.length).unwrap()); debug_assert!(snake.health <= crate::MAX_HEALTH); @@ -295,7 +295,7 @@ impl Battlesnake { self.health = self.health.saturating_sub(1); } - pub fn head(&self) -> &Coord { + #[must_use] pub fn head(&self) -> &Coord { debug_assert!(!self.body.is_empty()); self.body.front().expect("not empty") }