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

[position] Clean up Position's construction scheme

Browse source 
Position::default specifies the defaults for all field. Then, ::new() and
::starting() can use ..Default::default() in their implementations to avoid
having to specify empty values for all the internal structures.
This commit is contained in:
Eryn Wells 2024-02-02 07:28:12 -08:00
parent f09376f5dc
commit c8faad799e
1 changed files with 12 additions and 17 deletions
Download patch file Download diff file Expand all files Collapse all files

29
position/src/position/position.rs
View file

@ -26,16 +26,7 @@ pub struct Position {
impl Position { impl Position {
pub fn empty() -> Position { pub fn empty() -> Position {
Position { Default::default()
color_to_move: Color::White,
flags: Default::default(),
pieces: PieceBitBoards::default(),
en_passant_square: None,
sight: [OnceCell::new(), OnceCell::new()],
moves: OnceCell::new(),
half_move_counter: 0,
full_move_number: 1,
}
} }
/// Return a starting position. /// Return a starting position.
@ -60,13 +51,8 @@ impl Position {
Self { Self {
color_to_move: Color::White, color_to_move: Color::White,
flags: Flags::default(),
pieces: PieceBitBoards::new([white_pieces, black_pieces]), pieces: PieceBitBoards::new([white_pieces, black_pieces]),
en_passant_square: None, ..Default::default()
sight: [OnceCell::new(), OnceCell::new()],
moves: OnceCell::new(),
half_move_counter: 0,
full_move_number: 1,
} }
} }
@ -340,7 +326,16 @@ impl Position {
impl Default for Position { impl Default for Position {
fn default() -> Self { fn default() -> Self {
Self::empty() Self {
color_to_move: Color::White,
flags: Flags::default(),
pieces: PieceBitBoards::default(),
en_passant_square: None,
sight: [OnceCell::new(), OnceCell::new()],
moves: OnceCell::new(),
half_move_counter: 0,
full_move_number: 1,
}
} }
} }