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

[moves] Remove common suffix of attributes of AllPiecesMoveGenerator

Browse source 
Clippy complained that all the properties of the AllPiecesMoveGenerator struct
had the same suffix. It suggested I remove the _move_generator suffix. So I did.
This commit is contained in:
Eryn Wells 2025-06-07 19:36:51 -07:00
parent 8fd7ffa586
commit a8d83ad81d
1 changed files with 20 additions and 20 deletions
Download patch file Download diff file Expand all files Collapse all files

40
moves/src/generators/all.rs
View file

@ -10,23 +10,23 @@ use std::iter::{Fuse, FusedIterator};
#[must_use] #[must_use]
pub struct AllPiecesMoveGenerator { pub struct AllPiecesMoveGenerator {
pawn_move_generator: Fuse<PawnMoveGenerator>, pawn: Fuse<PawnMoveGenerator>,
knight_move_generator: Fuse<KnightMoveGenerator>, knight: Fuse<KnightMoveGenerator>,
bishop_move_generator: Fuse<BishopMoveGenerator>, bishop: Fuse<BishopMoveGenerator>,
rook_move_generator: Fuse<RookMoveGenerator>, rook: Fuse<RookMoveGenerator>,
queen_move_generator: Fuse<QueenMoveGenerator>, queen: Fuse<QueenMoveGenerator>,
king_move_generator: Fuse<KingMoveGenerator>, king: Fuse<KingMoveGenerator>,
} }
impl AllPiecesMoveGenerator { impl AllPiecesMoveGenerator {
pub fn new(board: &Board, color: Option<Color>) -> Self { pub fn new(board: &Board, color: Option<Color>) -> Self {
Self { Self {
pawn_move_generator: PawnMoveGenerator::new(board, color).fuse(), pawn: PawnMoveGenerator::new(board, color).fuse(),
knight_move_generator: KnightMoveGenerator::new(board, color).fuse(), knight: KnightMoveGenerator::new(board, color).fuse(),
bishop_move_generator: BishopMoveGenerator::new(board, color).fuse(), bishop: BishopMoveGenerator::new(board, color).fuse(),
rook_move_generator: RookMoveGenerator::new(board, color).fuse(), rook: RookMoveGenerator::new(board, color).fuse(),
queen_move_generator: QueenMoveGenerator::new(board, color).fuse(), queen: QueenMoveGenerator::new(board, color).fuse(),
king_move_generator: KingMoveGenerator::new(board, color).fuse(), king: KingMoveGenerator::new(board, color).fuse(),
} }
} }
} }
@ -36,12 +36,12 @@ impl Iterator for AllPiecesMoveGenerator {
fn next(&mut self) -> Option<Self::Item> { fn next(&mut self) -> Option<Self::Item> {
macro_rules! return_next_move { macro_rules! return_next_move {
($generator:expr) => {{ ($generator:expr) => {
let next_move = $generator.next(); let next_move = $generator.next();
if next_move.is_some() { if next_move.is_some() {
return next_move; return next_move;
} }
}}; };
} }
// All of these iterators are fused, meaning they are guaranteed to // All of these iterators are fused, meaning they are guaranteed to
@ -50,12 +50,12 @@ impl Iterator for AllPiecesMoveGenerator {
// generators' next() methods doesn't have to be repeated every time // generators' next() methods doesn't have to be repeated every time
// next() is called. // next() is called.
return_next_move!(self.pawn_move_generator); return_next_move!(self.pawn);
return_next_move!(self.knight_move_generator); return_next_move!(self.knight);
return_next_move!(self.bishop_move_generator); return_next_move!(self.bishop);
return_next_move!(self.rook_move_generator); return_next_move!(self.rook);
return_next_move!(self.queen_move_generator); return_next_move!(self.queen);
return_next_move!(self.king_move_generator); return_next_move!(self.king);
None None
} }