From 683d89b726c3a489bdf3e4420a0aed7d439b439a Mon Sep 17 00:00:00 2001 From: Eryn Wells Date: Sun, 21 Jan 2024 10:39:57 -0800 Subject: [PATCH] [board] Convert a few more Piece and PlacedPiece to pass by ref --- board/src/position/builders/move_builder.rs | 2 +- board/src/position/piece_sets.rs | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/board/src/position/builders/move_builder.rs b/board/src/position/builders/move_builder.rs index e899793..97802c1 100644 --- a/board/src/position/builders/move_builder.rs +++ b/board/src/position/builders/move_builder.rs @@ -156,7 +156,7 @@ impl<'p> Builder<'p, ValidatedMove> { if let Some(promotion) = promotion { pieces.remove_piece(&moving_piece); let _ = pieces - .place_piece(PlacedPiece::new(Piece::new(player, promotion), to_square)); + .place_piece(&PlacedPiece::new(Piece::new(player, promotion), to_square)); } else { pieces.move_piece(moving_piece.piece(), from_square, to_square); } diff --git a/board/src/position/piece_sets.rs b/board/src/position/piece_sets.rs index 07a1f03..cc01eda 100644 --- a/board/src/position/piece_sets.rs +++ b/board/src/position/piece_sets.rs @@ -74,13 +74,13 @@ impl PieceBitBoards { self.by_color_and_shape.bitboard_for_piece_mut(piece) } - pub(super) fn place_piece(&mut self, piece: PlacedPiece) -> Result<(), PlacePieceError> { + pub(super) fn place_piece(&mut self, piece: &PlacedPiece) -> Result<(), PlacePieceError> { self.place_piece_with_strategy(piece, Default::default()) } pub(super) fn place_piece_with_strategy( &mut self, - piece: PlacedPiece, + piece: &PlacedPiece, strategy: PlacePieceStrategy, ) -> Result<(), PlacePieceError> { let color = piece.color(); @@ -124,7 +124,7 @@ impl FromIterator for PieceBitBoards { let mut pieces = Self::default(); for piece in iter { - let _ = pieces.place_piece(piece); + let _ = pieces.place_piece(&piece); } pieces