Always have counterMoves associated
authorJoost VandeVondele <Joost.VandeVondele@gmail.com>
Thu, 9 Mar 2017 02:35:23 +0000 (18:35 -0800)
committerJoona Kiiski <joona@zoox.com>
Thu, 9 Mar 2017 02:35:23 +0000 (18:35 -0800)
commitd490bb99734bd6e2f8a0a352d2f3f1ba264ece66
treed2450468753d2e4dee313030e838a0cc4ef27319
parentcc76524c2e36345fcbf5fcdd3618a5dafd166b62
Always have counterMoves associated

Simplifies away all associated checks, leading to a ~0.5% speedup.
The code now explicitly checks if moves are OK, rather than using nullptr checks.

Verified for no regression:

LLR: 2.96 (-2.94,2.94) [-3.00,1.00]
Total: 32218 W: 5762 L: 5660 D: 20796

No functional change

Closes #1021
src/movepick.cpp
src/position.cpp
src/search.cpp