mirror of
https://github.com/sockspls/badfish
synced 2025-04-30 08:43:09 +00:00
Added some FIXME to track needed tests
This avoid us to forget some very needed tests now that futility has changed in a whole big chunk we need to fine tuning every splitted change. No functional change. Signed-off-by: Marco Costalba <mcostalba@gmail.com>
This commit is contained in:
parent
f37741cc83
commit
b95ba7b37e
1 changed files with 10 additions and 4 deletions
|
@ -1411,9 +1411,14 @@ namespace {
|
|||
update_gains(pos, ss[ply - 1].currentMove, ss[ply - 1].eval, ss[ply].eval);
|
||||
}
|
||||
|
||||
// Post futility pruning
|
||||
if (depth < SelectiveDepth && staticValue - PostFutilityValueMargin >= beta)
|
||||
return (staticValue - PostFutilityValueMargin);
|
||||
// Do a "stand pat". If we are above beta by a good margin then
|
||||
// return immediately.
|
||||
// FIXME: test with added condition 'allowNullmove || depth <= OnePly' and !value_is_mate(beta)
|
||||
// FIXME: test with modified condition 'depth < RazorDepth'
|
||||
if ( !isCheck
|
||||
&& depth < SelectiveDepth
|
||||
&& staticValue - PostFutilityValueMargin >= beta)
|
||||
return staticValue - PostFutilityValueMargin;
|
||||
|
||||
// Null move search
|
||||
if ( allowNullmove
|
||||
|
@ -1538,6 +1543,8 @@ namespace {
|
|||
movesSearched[moveCount++] = ss[ply].currentMove = move;
|
||||
|
||||
// Futility pruning for captures
|
||||
// FIXME: test disabling 'Futility pruning for captures'
|
||||
// FIXME: test with 'newDepth < RazorDepth'
|
||||
Color them = opposite_color(pos.side_to_move());
|
||||
|
||||
if ( !isCheck
|
||||
|
@ -1559,7 +1566,6 @@ namespace {
|
|||
continue;
|
||||
}
|
||||
|
||||
|
||||
// Futility pruning
|
||||
if ( !isCheck
|
||||
&& !dangerous
|
||||
|
|
Loading…
Add table
Reference in a new issue