From: Marco Costalba <mcostalba@gmail.com>
Date: Sun, 13 Jan 2013 13:28:22 +0000 (+0100)
Subject: Small change to "ponderhit" handling
X-Git-Url: https://git.sesse.net/?p=stockfish;a=commitdiff_plain;h=a66a7c3870d7b88054d0d6b7c7f734a60fab27f4;ds=sidebyside

Small change to "ponderhit" handling

Reset Limits.ponder only if search continue, but if
we are going to stop the search there is no need
(and is also confusing) to clear the 'ponder' flag.

This mimics the behaviour upon rceiving 'stop' when
pondering.

No functional change.
---

diff --git a/src/uci.cpp b/src/uci.cpp
index 49a0b22f..ddaad417 100644
--- a/src/uci.cpp
+++ b/src/uci.cpp
@@ -73,16 +73,18 @@ void UCI::loop(const string& args) {
       }
       else if (token == "ponderhit")
       {
-          // The opponent has played the expected move. GUI sends "ponderhit" if
-          // we were told to ponder on the same move the opponent has played. We
-          // should continue searching but switching from pondering to normal search.
-          Search::Limits.ponder = false;
-
+          // GUI sends "ponderhit" if we were told to ponder on the same move the
+          // opponent has played. In case Signals.stopOnPonderhit is set we are
+          // waiting for "ponderhit" to stop the search (for instance because we
+          // already ran out of time), otherwise we should continue searching but
+          // switching from pondering to normal search.
           if (Search::Signals.stopOnPonderhit)
           {
               Search::Signals.stop = true;
               Threads.main_thread()->wake_up(); // Could be sleeping
           }
+          else
+              Search::Limits.ponder = false;
       }
       else if (token == "perft" && (is >> token)) // Read perft depth
       {