From: Marco Costalba Date: Fri, 20 Feb 2015 09:49:51 +0000 (+0100) Subject: Use range-based-for in late join X-Git-Url: https://git.sesse.net/?p=stockfish;a=commitdiff_plain;h=a6f873cd8d05dda3638a7ea624ccbac90ecb98af;hp=40548c9153ea89c0b27b198efb443c5bb9b9c490 Use range-based-for in late join No functional change. --- diff --git a/src/search.cpp b/src/search.cpp index 04d67afe..bc41978f 100644 --- a/src/search.cpp +++ b/src/search.cpp @@ -1585,24 +1585,24 @@ void Thread::idle_loop() { Thread* bestThread = NULL; int bestScore = INT_MAX; - for (size_t i = 0; i < Threads.size(); ++i) + for (Thread* th : Threads) { - const size_t size = Threads[i]->splitPointsSize; // Local copy - sp = size ? &Threads[i]->splitPoints[size - 1] : nullptr; + const size_t size = th->splitPointsSize; // Local copy + sp = size ? &th->splitPoints[size - 1] : nullptr; if ( sp && sp->allSlavesSearching && sp->slavesMask.count() < MAX_SLAVES_PER_SPLITPOINT - && available_to(Threads[i])) + && available_to(th)) { - assert(this != Threads[i]); + assert(this != th); assert(!(this_sp && this_sp->slavesMask.none())); assert(Threads.size() > 2); // Prefer to join to SP with few parents to reduce the probability // that a cut-off occurs above us, and hence we waste our work. int level = -1; - for (SplitPoint* spp = Threads[i]->activeSplitPoint; spp; spp = spp->parentSplitPoint) + for (SplitPoint* spp = th->activeSplitPoint; spp; spp = spp->parentSplitPoint) level++; int score = level * 256 * 256 + (int)sp->slavesMask.count() * 256 - sp->depth * 1; @@ -1610,7 +1610,7 @@ void Thread::idle_loop() { if (score < bestScore) { bestSp = sp; - bestThread = Threads[i]; + bestThread = th; bestScore = score; } }