Explicitly pass RootMoves to TB probes
authorMarco Costalba <mcostalba@gmail.com>
Sun, 30 Nov 2014 13:59:09 +0000 (14:59 +0100)
committerJoona Kiiski <joona.kiiski@gmail.com>
Sat, 6 Dec 2014 15:08:21 +0000 (15:08 +0000)
commitba1464751d1186f723a2d2a5d18c06ddfc9a4cb3
treefa0d1621be85e61f23f20c62e3d1576e9d36d552
parenteeb6d923fa5e773ba223c0cede75705c1f3d9e89
Explicitly pass RootMoves to TB probes

Currently Search::RootMoves is accessed and even
modified by TB probing functions in a hidden
and sneaky way.

This is bad practice and makes the code tricky.
Instead explicily pass the vector as function
argument so to clarify that the vector is modified
inside the functions.

No functional change.
src/search.cpp
src/search.h
src/syzygy/tbcore.cpp
src/syzygy/tbprobe.cpp
src/syzygy/tbprobe.h