[board] Rename BitBoard::place_piece_at and BitBoard::remove_piece_at
→ set_square → clear_square
This commit is contained in:
		
							parent
							
								
									3ecc263701
								
							
						
					
					
						commit
						9ef53b76f5
					
				
					 2 changed files with 10 additions and 8 deletions
				
			
		| 
						 | 
					@ -45,7 +45,9 @@ impl BitBoard {
 | 
				
			||||||
    moves_getter!(rook_moves);
 | 
					    moves_getter!(rook_moves);
 | 
				
			||||||
    moves_getter!(queen_moves);
 | 
					    moves_getter!(queen_moves);
 | 
				
			||||||
    moves_getter!(king_moves);
 | 
					    moves_getter!(king_moves);
 | 
				
			||||||
 | 
					}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					impl BitBoard {
 | 
				
			||||||
    pub fn is_empty(&self) -> bool {
 | 
					    pub fn is_empty(&self) -> bool {
 | 
				
			||||||
        self.0 == 0
 | 
					        self.0 == 0
 | 
				
			||||||
    }
 | 
					    }
 | 
				
			||||||
| 
						 | 
					@ -54,12 +56,12 @@ impl BitBoard {
 | 
				
			||||||
        !(self & sq.into()).is_empty()
 | 
					        !(self & sq.into()).is_empty()
 | 
				
			||||||
    }
 | 
					    }
 | 
				
			||||||
 | 
					
 | 
				
			||||||
    pub fn place_piece_at(&mut self, sq: Square) {
 | 
					    pub fn set_square(&mut self, sq: Square) {
 | 
				
			||||||
        let sq_bb: BitBoard = sq.into();
 | 
					        let sq_bb: BitBoard = sq.into();
 | 
				
			||||||
        *self |= sq_bb
 | 
					        *self |= sq_bb
 | 
				
			||||||
    }
 | 
					    }
 | 
				
			||||||
 | 
					
 | 
				
			||||||
    fn remove_piece_at(&mut self, sq: Square) {
 | 
					    fn clear_square(&mut self, sq: Square) {
 | 
				
			||||||
        let sq_bb: BitBoard = sq.into();
 | 
					        let sq_bb: BitBoard = sq.into();
 | 
				
			||||||
        *self &= !sq_bb
 | 
					        *self &= !sq_bb
 | 
				
			||||||
    }
 | 
					    }
 | 
				
			||||||
| 
						 | 
					@ -224,18 +226,18 @@ mod tests {
 | 
				
			||||||
    }
 | 
					    }
 | 
				
			||||||
 | 
					
 | 
				
			||||||
    #[test]
 | 
					    #[test]
 | 
				
			||||||
    fn place_piece_at() {
 | 
					    fn set_square() {
 | 
				
			||||||
        let sq = Square::E4;
 | 
					        let sq = Square::E4;
 | 
				
			||||||
        let mut bb = BitBoard(0b1001100);
 | 
					        let mut bb = BitBoard(0b1001100);
 | 
				
			||||||
        bb.place_piece_at(sq);
 | 
					        bb.set_square(sq);
 | 
				
			||||||
        assert!(bb.has_piece_at(sq));
 | 
					        assert!(bb.has_piece_at(sq));
 | 
				
			||||||
    }
 | 
					    }
 | 
				
			||||||
 | 
					
 | 
				
			||||||
    #[test]
 | 
					    #[test]
 | 
				
			||||||
    fn remove_piece_at() {
 | 
					    fn clear_square() {
 | 
				
			||||||
        let sq = Square::A3;
 | 
					        let sq = Square::A3;
 | 
				
			||||||
        let mut bb = BitBoard(0b1001100);
 | 
					        let mut bb = BitBoard(0b1001100);
 | 
				
			||||||
        bb.remove_piece_at(sq);
 | 
					        bb.clear_square(sq);
 | 
				
			||||||
        assert!(!bb.has_piece_at(sq));
 | 
					        assert!(!bb.has_piece_at(sq));
 | 
				
			||||||
    }
 | 
					    }
 | 
				
			||||||
 | 
					
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
| 
						 | 
					@ -141,10 +141,10 @@ impl Position {
 | 
				
			||||||
            return Err(PiecePlacementError::ExistsOnSquare);
 | 
					            return Err(PiecePlacementError::ExistsOnSquare);
 | 
				
			||||||
        }
 | 
					        }
 | 
				
			||||||
 | 
					
 | 
				
			||||||
        type_bb.place_piece_at(square);
 | 
					        type_bb.set_square(square);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
        let color_bb = &mut self.bitboard_for_color_mut(piece.color());
 | 
					        let color_bb = &mut self.bitboard_for_color_mut(piece.color());
 | 
				
			||||||
        color_bb.place_piece_at(square);
 | 
					        color_bb.set_square(square);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
        Ok(())
 | 
					        Ok(())
 | 
				
			||||||
    }
 | 
					    }
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
		Loading…
	
	Add table
		Add a link
		
	
		Reference in a new issue