Re-apply the fix for Limits::ponder race

But this time correctly set Threads.ponder

We avoid using 'limits' for passing pondering
flag because we don't want to have 2 ponder
variables in search scope: Search::Limits.ponder
and Threads.ponder. This would be confusing also
because limits.ponder is set at the beginning of
the search and never changes, instead Threads.ponder
can change value asynchronously during search.

No functional change.
This commit is contained in:
Marco Costalba
2017-08-10 12:32:50 -07:00
parent 44236f4ed9
commit 66c5eaebd8
5 changed files with 19 additions and 18 deletions

View File

@@ -96,12 +96,12 @@ struct ThreadPool : public std::vector<Thread*> {
void exit(); // be initialized and valid during the whole thread lifetime.
MainThread* main() { return static_cast<MainThread*>(at(0)); }
void start_thinking(Position&, StateListPtr&, const Search::LimitsType&);
void start_thinking(Position&, StateListPtr&, const Search::LimitsType&, bool = false);
void read_uci_options();
uint64_t nodes_searched() const;
uint64_t tb_hits() const;
std::atomic_bool stop, stopOnPonderhit;
std::atomic_bool stop, ponder, stopOnPonderhit;
private:
StateListPtr setupStates;