eryn/chessfriend
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

[board] Pass piece argument to Position::bitboard_for_piece() by value instead of by reference

Browse source
This commit is contained in:
Eryn Wells 2023-12-27 08:31:02 -07:00
parent 4bec1538a1
commit 50a6be2f72
2 changed files with 3 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

2
board/src/position/pieces.rs
View file

@ -45,7 +45,7 @@ impl<'a> Iterator for Pieces<'a> {
while let Some(shape) = self.shape_iterator.next() {
let piece = Piece::new(self.color, shape);
let bitboard = self.position.bitboard_for_piece(&piece);
let bitboard = self.position.bitboard_for_piece(piece);
if bitboard.is_empty() {
println!("No {:?} pieces; looping", &piece);
continue;

4
board/src/position/position.rs
View file

@ -90,7 +90,7 @@ impl Position {
Ok(())
}
pub(super) fn bitboard_for_piece(&self, piece: &Piece) -> &BitBoard {
pub(super) fn bitboard_for_piece(&self, piece: Piece) -> &BitBoard {
&self.pieces_per_type[piece.color as usize][piece.shape as usize]
}
@ -102,7 +102,7 @@ impl Position {
for color in Color::iter() {
for shape in Shape::iter() {
let piece = Piece::new(color, shape);
let bb = self.bitboard_for_piece(&piece);
let bb = self.bitboard_for_piece(piece);
if bb.has_piece_at(&sq) {
return Some(PlacedPiece::new(piece, sq));
}