From: Raminder Singh <romi_ssk@yahoo.co.in>
Date: Wed, 25 Sep 2013 13:17:59 +0000 (+0530)
Subject: Fix best move lookup bug
X-Git-Url: https://git.sesse.net/?p=stockfish;a=commitdiff_plain;h=e654209211a1bcd40a0ff00c5bdba284890ea857;hp=b742a3f29aac762616537eb032db4a2455df6b96

Fix best move lookup bug

If the uci option 'Best Book Move' is set to true the lookup still
returns a move at random instead of the move with the highest
weight.

No functional change.
---

diff --git a/src/book.cpp b/src/book.cpp
index c5c40dde..ec0a0f83 100644
--- a/src/book.cpp
+++ b/src/book.cpp
@@ -413,7 +413,7 @@ Move PolyglotBook::probe(const Position& pos, const string& fName, bool pickBest
       // Choose book move according to its score. If a move has a very
       // high score it has higher probability to be choosen than a move
       // with lower score. Note that first entry is always chosen.
-      if (   (sum && rkiss.rand<unsigned>() % sum < e.count)
+      if (   (!pickBest && sum && rkiss.rand<unsigned>() % sum < e.count)
           || (pickBest && e.count == best))
           move = Move(e.move);
   }