[position] Pass the PlacedPiece in move_set_for_piece() by reference to all the MoveGenerators
This commit is contained in:
		
							parent
							
								
									29eb56a5d7
								
							
						
					
					
						commit
						41c3a2075c
					
				
					 7 changed files with 16 additions and 16 deletions
				
			
		|  | @ -110,7 +110,7 @@ trait MoveGeneratorInternal { | |||
|                 .map(|square| { | ||||
|                     let piece = PlacedPiece::new(piece, square); | ||||
|                     let move_set = | ||||
|                         Self::move_set_for_piece(position, piece, capture_mask, push_mask); | ||||
|                         Self::move_set_for_piece(position, &piece, capture_mask, push_mask); | ||||
|                     (square, move_set) | ||||
|                 }), | ||||
|         ) | ||||
|  | @ -118,7 +118,7 @@ trait MoveGeneratorInternal { | |||
| 
 | ||||
|     fn move_set_for_piece( | ||||
|         position: &Position, | ||||
|         piece: PlacedPiece, | ||||
|         piece: &PlacedPiece, | ||||
|         capture_mask: BitBoard, | ||||
|         push_mask: BitBoard, | ||||
|     ) -> MoveSet; | ||||
|  |  | |||
|  | @ -14,7 +14,7 @@ impl MoveGeneratorInternal for ClassicalMoveGenerator { | |||
| 
 | ||||
|     fn move_set_for_piece( | ||||
|         position: &Position, | ||||
|         piece: PlacedPiece, | ||||
|         piece: &PlacedPiece, | ||||
|         capture_mask: BitBoard, | ||||
|         push_mask: BitBoard, | ||||
|     ) -> MoveSet { | ||||
|  | @ -49,7 +49,7 @@ impl MoveGeneratorInternal for ClassicalMoveGenerator { | |||
|         let quiet_moves = all_moves & (empty_squares | !friendly_pieces) & push_mask; | ||||
|         let capture_moves = all_moves & opposing_pieces & capture_mask; | ||||
| 
 | ||||
|         MoveSet::new(piece) | ||||
|         MoveSet::new(*piece) | ||||
|             .quiet_moves(quiet_moves) | ||||
|             .capture_moves(capture_moves) | ||||
|     } | ||||
|  |  | |||
|  | @ -19,7 +19,7 @@ impl MoveGeneratorInternal for KingMoveGenerator { | |||
| 
 | ||||
|     fn move_set_for_piece( | ||||
|         position: &Position, | ||||
|         placed_piece: PlacedPiece, | ||||
|         placed_piece: &PlacedPiece, | ||||
|         _capture_mask: BitBoard, | ||||
|         _push_mask: BitBoard, | ||||
|     ) -> MoveSet { | ||||
|  | @ -39,7 +39,7 @@ impl MoveGeneratorInternal for KingMoveGenerator { | |||
|         let quiet_moves = all_king_moves & safe_empty_squares; | ||||
|         let capture_moves = all_king_moves & opposing_pieces_on_safe_squares; | ||||
| 
 | ||||
|         let mut move_set = MoveSet::new(placed_piece) | ||||
|         let mut move_set = MoveSet::new(*placed_piece) | ||||
|             .quiet_moves(quiet_moves) | ||||
|             .capture_moves(capture_moves); | ||||
| 
 | ||||
|  |  | |||
|  | @ -14,7 +14,7 @@ impl MoveGeneratorInternal for KnightMoveGenerator { | |||
| 
 | ||||
|     fn move_set_for_piece( | ||||
|         position: &Position, | ||||
|         placed_piece: PlacedPiece, | ||||
|         placed_piece: &PlacedPiece, | ||||
|         capture_mask: BitBoard, | ||||
|         push_mask: BitBoard, | ||||
|     ) -> MoveSet { | ||||
|  | @ -25,7 +25,7 @@ impl MoveGeneratorInternal for KnightMoveGenerator { | |||
|         let quiet_moves = knight_moves & empty_squares & push_mask; | ||||
|         let capture_moves = knight_moves & opposing_pieces & capture_mask; | ||||
| 
 | ||||
|         MoveSet::new(placed_piece) | ||||
|         MoveSet::new(*placed_piece) | ||||
|             .quiet_moves(quiet_moves) | ||||
|             .capture_moves(capture_moves) | ||||
|     } | ||||
|  |  | |||
|  | @ -17,14 +17,14 @@ impl MoveGeneratorInternal for PawnMoveGenerator { | |||
| 
 | ||||
|     fn move_set_for_piece( | ||||
|         position: &Position, | ||||
|         placed_piece: PlacedPiece, | ||||
|         placed_piece: &PlacedPiece, | ||||
|         capture_mask: BitBoard, | ||||
|         push_mask: BitBoard, | ||||
|     ) -> MoveSet { | ||||
|         let capture_moves = Self::attacks(position, placed_piece) & capture_mask; | ||||
|         let quiet_moves = Self::pushes(position, placed_piece) & push_mask; | ||||
|         let capture_moves = Self::attacks(position, &placed_piece) & capture_mask; | ||||
|         let quiet_moves = Self::pushes(position, &placed_piece) & push_mask; | ||||
| 
 | ||||
|         MoveSet::new(placed_piece) | ||||
|         MoveSet::new(*placed_piece) | ||||
|             .quiet_moves(quiet_moves) | ||||
|             .capture_moves(capture_moves) | ||||
|     } | ||||
|  |  | |||
|  | @ -14,7 +14,7 @@ impl MoveGeneratorInternal for ClassicalMoveGenerator { | |||
| 
 | ||||
|     fn move_set_for_piece( | ||||
|         position: &Position, | ||||
|         placed_piece: PlacedPiece, | ||||
|         placed_piece: &PlacedPiece, | ||||
|         capture_mask: BitBoard, | ||||
|         push_mask: BitBoard, | ||||
|     ) -> MoveSet { | ||||
|  | @ -56,7 +56,7 @@ impl MoveGeneratorInternal for ClassicalMoveGenerator { | |||
|         let quiet_moves = all_moves & (empty_squares | !friendly_pieces) & push_mask; | ||||
|         let capture_moves = all_moves & opposing_pieces & capture_mask; | ||||
| 
 | ||||
|         MoveSet::new(placed_piece) | ||||
|         MoveSet::new(*placed_piece) | ||||
|             .quiet_moves(quiet_moves) | ||||
|             .capture_moves(capture_moves) | ||||
|     } | ||||
|  |  | |||
|  | @ -14,7 +14,7 @@ impl MoveGeneratorInternal for ClassicalMoveGenerator { | |||
| 
 | ||||
|     fn move_set_for_piece( | ||||
|         position: &Position, | ||||
|         placed_piece: PlacedPiece, | ||||
|         placed_piece: &PlacedPiece, | ||||
|         capture_mask: BitBoard, | ||||
|         push_mask: BitBoard, | ||||
|     ) -> MoveSet { | ||||
|  | @ -52,7 +52,7 @@ impl MoveGeneratorInternal for ClassicalMoveGenerator { | |||
|         let quiet_moves = all_moves & (empty_squares | !friendly_pieces) & capture_mask; | ||||
|         let capture_moves = all_moves & opposing_pieces & push_mask; | ||||
| 
 | ||||
|         MoveSet::new(placed_piece) | ||||
|         MoveSet::new(*placed_piece) | ||||
|             .quiet_moves(quiet_moves) | ||||
|             .capture_moves(capture_moves) | ||||
|     } | ||||
|  |  | |||
		Loading…
	
	Add table
		Add a link
		
	
		Reference in a new issue