From: Marco Costalba Date: Wed, 10 Aug 2011 11:05:33 +0000 (+0100) Subject: Remove Thread::WORKISWAITING X-Git-Url: https://git.sesse.net/?p=stockfish;a=commitdiff_plain;h=c386ce0023f114e387c02a7e36256d8b0a908cd0 Remove Thread::WORKISWAITING Set the state directly to Thread::SEARCHING No functional change. Signed-off-by: Marco Costalba --- diff --git a/src/search.cpp b/src/search.cpp index 9923ef1d..65b128f5 100644 --- a/src/search.cpp +++ b/src/search.cpp @@ -2187,12 +2187,10 @@ void Thread::idle_loop(SplitPoint* sp) { } // If this thread has been assigned work, launch a search - if (state == Thread::WORKISWAITING) + if (state == Thread::SEARCHING) { assert(!do_terminate); - state = Thread::SEARCHING; - // Copy split point position and search stack and call search() SearchStack ss[PLY_MAX_PLUS_2]; SplitPoint* tsp = splitPoint; diff --git a/src/thread.cpp b/src/thread.cpp index ac0be89f..f620c71c 100644 --- a/src/thread.cpp +++ b/src/thread.cpp @@ -303,7 +303,7 @@ Value ThreadsManager::split(Position& pos, SearchStack* ss, Value alpha, Value b threads[i].splitPoint = sp; // This makes the slave to exit from idle_loop() - threads[i].state = Thread::WORKISWAITING; + threads[i].state = Thread::SEARCHING; if (useSleepingThreads) threads[i].wake_up(); @@ -317,7 +317,6 @@ Value ThreadsManager::split(Position& pos, SearchStack* ss, Value alpha, Value b masterThread.splitPoint = sp; masterThread.activeSplitPoints++; - masterThread.state = Thread::WORKISWAITING; // Everything is set up. The master thread enters the idle loop, from // which it will instantly launch a search, because its state is diff --git a/src/thread.h b/src/thread.h index 574c5964..8d3fd615 100644 --- a/src/thread.h +++ b/src/thread.h @@ -68,7 +68,6 @@ struct Thread { enum ThreadState { AVAILABLE, // Thread is waiting for work - WORKISWAITING, // Master has ordered us to start searching SEARCHING // Thread is performing work };