From 63a04134d0841bb362f42d600faf614038fff494 Mon Sep 17 00:00:00 2001 From: Marco Costalba Date: Sun, 25 Sep 2011 16:28:38 +0100 Subject: [PATCH 1/1] 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 --- src/search.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) 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)); -- 2.39.2