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

Micro optimize pl_move_is_legal()

Browse source 
No functional change.

Signed-off-by: Marco Costalba <mcostalba@gmail.com>
This commit is contained in:
Marco Costalba 2009-03-07 20:08:45 +01:00
parent 72b88e09e1
commit 964bd86272
1 changed files with 4 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
src/position.cpp
View file

@ -479,7 +479,6 @@ bool Position::pl_move_is_legal(Move m, Bitboard pinned) const {
return true; return true;
Color us = side_to_move(); Color us = side_to_move();
Color them = opposite_color(us);
Square from = move_from(m); Square from = move_from(m);
Square ksq = king_square(us); Square ksq = king_square(us);
@ -491,6 +490,7 @@ bool Position::pl_move_is_legal(Move m, Bitboard pinned) const {
// after the move is made // after the move is made
if (move_is_ep(m)) if (move_is_ep(m))
{ {
Color them = opposite_color(us);
Square to = move_to(m); Square to = move_to(m);
Square capsq = make_square(square_file(to), square_rank(from)); Square capsq = make_square(square_file(to), square_rank(from));
Bitboard b = occupied_squares(); Bitboard b = occupied_squares();
@ -511,11 +511,12 @@ bool Position::pl_move_is_legal(Move m, Bitboard pinned) const {
// If the moving piece is a king, check whether the destination // If the moving piece is a king, check whether the destination
// square is attacked by the opponent. // square is attacked by the opponent.
if (from == ksq) if (from == ksq)
return !(square_is_attacked(move_to(m), them)); return !(square_is_attacked(move_to(m), opposite_color(us)));
// A non-king move is legal if and only if it is not pinned or it // A non-king move is legal if and only if it is not pinned or it
// is moving along the ray towards or away from the king. // is moving along the ray towards or away from the king.
return ( !bit_is_set(pinned, from) return ( !pinned
|| !bit_is_set(pinned, from)
|| (direction_between_squares(from, ksq) == direction_between_squares(move_to(m), ksq))); || (direction_between_squares(from, ksq) == direction_between_squares(move_to(m), ksq)));
} }