From: Marco Costalba <mcostalba@gmail.com>
Date: Tue, 17 May 2011 22:44:55 +0000 (+0100)
Subject: Micro-optimize pl_move_is_legal()
X-Git-Url: https://git.sesse.net/?p=stockfish;a=commitdiff_plain;h=89a06f6651882248e185bb597135af3249ec8f6a;ds=sidebyside

Micro-optimize pl_move_is_legal()

Remove the check for castling moves because it is
already implicit in the check for king moves and castling
is so rare that doing the check is just a slow down.

Thanks to Marek Kwiatkowski.

No functional change.

Signed-off-by: Marco Costalba <mcostalba@gmail.com>
---

diff --git a/src/position.cpp b/src/position.cpp
index a9c98626..59caa88e 100644
--- a/src/position.cpp
+++ b/src/position.cpp
@@ -570,10 +570,6 @@ bool Position::pl_move_is_legal(Move m, Bitboard pinned) const {
   assert(move_is_ok(m));
   assert(pinned == pinned_pieces(side_to_move()));
 
-  // Castling moves are checked for legality during move generation.
-  if (move_is_castle(m))
-      return true;
-
   // En passant captures are a tricky special case. Because they are
   // rather uncommon, we do it simply by testing whether the king is attacked
   // after the move is made