No need to test for MOVE_NONE before move_is_ok()
authorMarco Costalba <mcostalba@gmail.com>
Sun, 25 Sep 2011 15:28:38 +0000 (16:28 +0100)
committerMarco Costalba <mcostalba@gmail.com>
Sun, 25 Sep 2011 15:30:29 +0000 (16:30 +0100)
Function move_is_ok() already catches the move == MOVE_NONE case.

No functional change.

Signed-off-by: Marco Costalba <mcostalba@gmail.com>
src/search.cpp

index 0e3ba85c82ae5bf421ce7192d8e03a621d636979..ed18936f760eb71cb7450daa8c12279f7d3f1ef0 100644 (file)
@@ -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));