From 09fbe1be22dbab282323632bf46873b9c37b55d5 Mon Sep 17 00:00:00 2001 From: Eryn Wells Date: Mon, 2 Jun 2025 15:46:10 -0700 Subject: [PATCH] [board] Make Board::pieces private Do not allow direct access to the internal piece set. Update call sites to use Board API instead. --- board/src/board.rs | 2 +- board/src/check.rs | 2 +- board/src/fen.rs | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/board/src/board.rs b/board/src/board.rs index 4e3a0f6..3e866e0 100644 --- a/board/src/board.rs +++ b/board/src/board.rs @@ -15,7 +15,7 @@ pub type FullMoveClock = u32; #[derive(Clone, Debug, Default, Eq, PartialEq)] pub struct Board { pub active_color: Color, - pub pieces: PieceSet, + pieces: PieceSet, pub castling_rights: castle::Rights, pub en_passant_target: Option, pub half_move_clock: HalfMoveClock, diff --git a/board/src/check.rs b/board/src/check.rs index 8e6246d..6873cf4 100644 --- a/board/src/check.rs +++ b/board/src/check.rs @@ -23,7 +23,7 @@ impl Board { } fn king_bitboard(&self, color: Color) -> BitBoard { - self.pieces.find_pieces(Piece::king(color)) + self.find_pieces(Piece::king(color)) } } diff --git a/board/src/fen.rs b/board/src/fen.rs index 657f87b..42d5656 100644 --- a/board/src/fen.rs +++ b/board/src/fen.rs @@ -209,7 +209,7 @@ impl FromFenStr for Board { let file = files.next().ok_or(FromFenStrError::MissingPlacement)?; let piece = Piece::from_fen_str(&ch.to_string())?; - let _ = board.pieces.place( + let _ = board.place_piece( piece, Square::from_file_rank(*file, *rank), PlacePieceStrategy::default(),