kate/BadFish
1
0
Fork 0
mirror of https://github.com/sockspls/badfish synced 2025-05-03 01:59:36 +00:00
Code Issues Releases Wiki Activity

Simplify the promotion case of move_gives_check()

Browse source 
No functional change.

Signed-off-by: Marco Costalba <mcostalba@gmail.com>
This commit is contained in:
Marco Costalba 2011-10-22 14:53:35 +01:00
parent 23943208ec
commit b5bbc1f713
1 changed files with 4 additions and 14 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

18
src/position.cpp
View file

@ -618,6 +618,9 @@ bool Position::is_pseudo_legal(const Move m) const {
else if (!bit_is_set(attacks_from(pc, from), to)) else if (!bit_is_set(attacks_from(pc, from), to))
return false; return false;
// Evasions generator already takes care to avoid some kind of illegal moves
// and pl_move_is_legal() relies on this. So we have to take care that the
// same kind of moves are filtered out here.
if (in_check()) if (in_check())
{ {
// In case of king moves under check we have to remove king so to catch // In case of king moves under check we have to remove king so to catch
@ -685,20 +688,7 @@ bool Position::move_gives_check(Move m, const CheckInfo& ci) const {
if (is_promotion(m)) if (is_promotion(m))
{ {
clear_bit(&b, from); clear_bit(&b, from);
return bit_is_set(attacks_from(Piece(promotion_piece_type(m)), to, b), ksq);
switch (promotion_piece_type(m))
{
case KNIGHT:
return bit_is_set(attacks_from<KNIGHT>(to), ksq);
case BISHOP:
return bit_is_set(bishop_attacks_bb(to, b), ksq);
case ROOK:
return bit_is_set(rook_attacks_bb(to, b), ksq);
case QUEEN:
return bit_is_set(queen_attacks_bb(to, b), ksq);
default:
assert(false);
}
} }
// En passant capture with check ? We have already handled the case // En passant capture with check ? We have already handled the case