The patch was tested for correctness by running bench with and
without the change against current master, and the tablebase hit
numbers were found to be identical in both cases. See the pull
request comments for details:
https://github.com/official-stockfish/Stockfish/pull/1826
No functional change.
continue; // First on diagonal, second above
else if (!off_A1H8(s1) && !off_A1H8(s2))
- bothOnDiagonal.push_back(std::make_pair(idx, s2));
+ bothOnDiagonal.emplace_back(idx, s2);
else
MapKK[idx][s2] = code++;
for (size_t idx = 0; idx < Options["Threads"]; idx++)
{
- threads.push_back(std::thread([this, idx]() {
+ threads.emplace_back([this, idx]() {
// Thread binding gives faster search on systems with a first-touch policy
if (Options["Threads"] > 8)
stride : clusterCount - start;
std::memset(&table[start], 0, len * sizeof(Cluster));
- }));
+ });
}
for (std::thread& th: threads)