From 1ee1d852fe5f82b73a9799db7cdd8468c0fa5faa Mon Sep 17 00:00:00 2001 From: Marco Costalba Date: Tue, 5 Oct 2010 22:10:42 +0100 Subject: [PATCH] Skip an useless compare in space evaluation Spotted by Marek Kwiatkowski. No functional change. Signed-off-by: Marco Costalba --- src/evaluate.cpp | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/evaluate.cpp b/src/evaluate.cpp index 86cebecb..041ce86f 100644 --- a/src/evaluate.cpp +++ b/src/evaluate.cpp @@ -331,10 +331,8 @@ Value do_evaluate(const Position& pos, Value& margin) { bonus += evaluate_passed_pawns(pos, ei) - evaluate_passed_pawns(pos, ei); - Phase phase = mi->game_phase(); - // Evaluate space for both sides, only in middle-game. - if (phase > PHASE_ENDGAME && mi->space_weight() > 0) + if (mi->space_weight()) { int s = evaluate_space(pos, ei) - evaluate_space(pos, ei); bonus += apply_weight(make_score(s * mi->space_weight(), 0), Weights[Space]); @@ -343,6 +341,7 @@ Value do_evaluate(const Position& pos, Value& margin) { // Scale winning side if position is more drawish that what it appears ScaleFactor sf = eg_value(bonus) > VALUE_ZERO ? mi->scale_factor(pos, WHITE) : mi->scale_factor(pos, BLACK); + Phase phase = mi->game_phase(); // If we don't already have an unusual scale factor, check for opposite // colored bishop endgames, and use a lower scale for those. -- 2.39.2