From: Marco Costalba <mcostalba@gmail.com>
Date: Thu, 6 Jan 2011 12:33:40 +0000 (+0100)
Subject: Do not make any assumption on the move in move_is_legal()
X-Git-Url: https://git.sesse.net/?p=stockfish;a=commitdiff_plain;h=2e46db436965e9f3c9c24d028cc6a8bfc85303d4

Do not make any assumption on the move in move_is_legal()

We must be able to filter out also moves where move_is_ok()
is false.

And actually we are. Tested on all the default position injecting
a number from -1000000 to 1000000 casted to a Move.

No functional change.

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

diff --git a/src/movegen.cpp b/src/movegen.cpp
index 0f3e4ddb..8f0a825f 100644
--- a/src/movegen.cpp
+++ b/src/movegen.cpp
@@ -386,7 +386,6 @@ bool move_is_legal(const Position& pos, const Move m) {
 bool move_is_legal(const Position& pos, const Move m, Bitboard pinned) {
 
   assert(pos.is_ok());
-  assert(move_is_ok(m));
   assert(pinned == pos.pinned_pieces(pos.side_to_move()));
 
   Color us = pos.side_to_move();