stefiosif/zeal
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Fix fen parsing for castling rights

Browse Source
This commit is contained in:
stefiosif
2024-06-29 16:21:39 +03:00
parent 349007a445
commit b46c5874db
1 changed files with 9 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
src/fen.rs
View File

@@ -97,9 +97,9 @@ fn castling_ability(castling: &str) -> Result<[Castle; 2], FenError> {
let mut castling_ability: [Castle; 2] = [Castle::None, Castle::None]; let mut castling_ability: [Castle; 2] = [Castle::None, Castle::None];
let white_king_and_queen = bitflag & 0b11 == 0b11; let white_king_and_queen = (bitflag >> 2) & 0b11 == 0b11;
let white_king = (bitflag >> 1) & 1 == 1; let white_king = (bitflag >> 3) & 1 == 1;
let white_queen = bitflag & 1 == 1; let white_queen = (bitflag >> 2) & 1 == 1;
castling_ability[0] = match (white_king_and_queen, white_king, white_queen) { castling_ability[0] = match (white_king_and_queen, white_king, white_queen) {
(true, _, _) => Castle::Both, (true, _, _) => Castle::Both,
(_, true, _) => Castle::Short, (_, true, _) => Castle::Short,
@@ -107,9 +107,9 @@ fn castling_ability(castling: &str) -> Result<[Castle; 2], FenError> {
_ => Castle::None, _ => Castle::None,
}; };
let black_king_and_queen = (bitflag >> 2) & 0b11 == 0b11; let black_king_and_queen = bitflag & 0b11 == 0b11;
let black_king = (bitflag >> 3) & 1 == 1; let black_king = (bitflag >> 1) & 1 == 1;
let black_queen = (bitflag >> 2) & 1 == 1; let black_queen = bitflag & 1 == 1;
castling_ability[1] = match (black_king_and_queen, black_king, black_queen) { castling_ability[1] = match (black_king_and_queen, black_king, black_queen) {
(true, _, _) => Castle::Both, (true, _, _) => Castle::Both,
(_, true, _) => Castle::Short, (_, true, _) => Castle::Short,
@@ -197,4 +197,7 @@ mod tests {
fn test_fen_error() -> () { fn test_fen_error() -> () {
from_fen(FEN_EXAMPLE[4]).unwrap(); from_fen(FEN_EXAMPLE[4]).unwrap();
} }
//TODO: add more happy path scenarios
//TODO: test each panic e.g. #[should_panic(expected = "less than or equal to 100")]
} }