From: Marco Costalba <mcostalba@gmail.com>
Date: Sun, 25 Sep 2011 15:28:38 +0000 (+0100)
Subject: No need to test for MOVE_NONE before move_is_ok()
X-Git-Url: https://git.sesse.net/?p=stockfish;a=commitdiff_plain;h=63a04134d0841bb362f42d600faf614038fff494

No need to test for MOVE_NONE before move_is_ok()

Function move_is_ok() already catches the move == MOVE_NONE case.

No functional change.

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

diff --git a/src/search.cpp b/src/search.cpp
index 0e3ba85c..ed18936f 100644
--- a/src/search.cpp
+++ b/src/search.cpp
@@ -1549,8 +1549,8 @@ split_point_start: // At split points actual search starts from here
     Piece p1, p2;
     Square ksq;
 
-    assert(m1 && move_is_ok(m1));
-    assert(m2 && move_is_ok(m2));
+    assert(move_is_ok(m1));
+    assert(move_is_ok(m2));
 
     // Case 1: The moving piece is the same in both moves
     f2 = move_from(m2);
@@ -1626,7 +1626,7 @@ split_point_start: // At split points actual search starts from here
   bool connected_threat(const Position& pos, Move m, Move threat) {
 
     assert(move_is_ok(m));
-    assert(threat && move_is_ok(threat));
+    assert(move_is_ok(threat));
     assert(!pos.move_is_capture_or_promotion(m));
     assert(!pos.move_is_passed_pawn_push(m));