Remove Stack/thread dependence in movepick
authorJoost VandeVondele <Joost.VandeVondele@gmail.com>
Fri, 30 Jun 2017 15:20:00 +0000 (17:20 +0200)
committerMarco Costalba <mcostalba@gmail.com>
Sun, 6 Aug 2017 08:45:54 +0000 (01:45 -0700)
as a lower level routine, movepicker should not depend on the
search stack or the thread class, removing a circular dependency.
Instead of copying the search stack into the movepicker object,
as well as accessing the thread class for one of the histories,
pass the required fields explicitly to the constructor (removing
the need for thread.h and implicitly search.h in movepick.cpp).
The signature is thus longer, but more explicit:

Also some renaming of histories structures while there.

passed STC [-3,1], suggesting a small elo impact:

LLR: 3.13 (-2.94,2.94) [-3.00,1.00]
Total: 381053 W: 68071 L: 68551 D: 244431
elo =   -0.438 +-    0.660 LOS:    9.7%

No functional change.


No differences found