Fix errouneus reset of ss->threatMove
authorMarco Costalba <mcostalba@gmail.com>
Fri, 23 Jul 2010 01:26:10 +0000 (02:26 +0100)
committerMarco Costalba <mcostalba@gmail.com>
Fri, 23 Jul 2010 01:26:57 +0000 (02:26 +0100)
commitd004ec924de058f46076b23c0cf244d46e752ac7
treebe900ba510a8aa69d86a4fb85030cb59e1e4cf31
parent5c3aeae566081f9222280b3e02e679a38bede5cf
Fix errouneus reset of ss->threatMove

After we set ss->threatMove we could go under a IID step that
resets SearchStack ss and so also ss->threatMove.

When later we use that field in futility pruning we have this
set to MOVE_NONE !

The fix is to use a local variable and add threatMove to SplitPoint
to pass this move to slaves.

Spotted by Ralph Stoesser, fix suggested by Richard Vida.

Signed-off-by: Marco Costalba <mcostalba@gmail.com>
src/search.cpp
src/search.h
src/thread.h