X-Git-Url: https://git.sesse.net/?p=stockfish;a=blobdiff_plain;f=src%2Fmovepick.cpp;h=a604fe14bd72efd81b065e38743914d9b4f154fe;hp=5c8338c0c19ab96d08f55879015d8f422002985c;hb=19dd0de4ff83e682bafa56d27f302d8f00d3040d;hpb=5a156df719790ddd882a928ddf721f8ba81587b1 diff --git a/src/movepick.cpp b/src/movepick.cpp index 5c8338c0..a604fe14 100644 --- a/src/movepick.cpp +++ b/src/movepick.cpp @@ -1,7 +1,7 @@ /* Stockfish, a UCI chess playing engine derived from Glaurung 2.1 Copyright (C) 2004-2008 Tord Romstad (Glaurung author) - Copyright (C) 2008-2012 Marco Costalba, Joona Kiiski, Tord Romstad + Copyright (C) 2008-2013 Marco Costalba, Joona Kiiski, Tord Romstad Stockfish is free software: you can redistribute it and/or modify it under the terms of the GNU General Public License as published by @@ -70,7 +70,7 @@ namespace { /// search captures, promotions and some checks) and about how important good /// move ordering is at the current node. -MovePicker::MovePicker(const Position& p, Move ttm, Depth d, const History& h, +MovePicker::MovePicker(const Position& p, Move ttm, Depth d, const History& h, const Refutations& r, Search::Stack* s, Value beta) : pos(p), Hist(h), depth(d) { assert(d > DEPTH_ZERO); @@ -89,6 +89,8 @@ MovePicker::MovePicker(const Position& p, Move ttm, Depth d, const History& h, killers[0].move = ss->killers[0]; killers[1].move = ss->killers[1]; + Square prevSq = to_sq((ss-1)->currentMove); + killers[2].move = r[pos.piece_on(prevSq)][prevSq]; // Consider sligtly negative captures as good if at low depth and far from beta if (ss && ss->staticEval < beta - PawnValueMg && d < 3 * ONE_PLY) @@ -237,7 +239,7 @@ void MovePicker::generate_next() { case KILLERS_S1: cur = killers; - end = cur + 2; + end = cur + 3; return; case QUIETS_1_S1: @@ -329,7 +331,8 @@ Move MovePicker::next_move() { move = (cur++)->move; if ( move != ttMove && move != killers[0].move - && move != killers[1].move) + && move != killers[1].move + && move != killers[2].move) return move; break; @@ -374,4 +377,4 @@ Move MovePicker::next_move() { /// from the split point's shared MovePicker object. This function is not thread /// safe so must be lock protected by the caller. template<> -Move MovePicker::next_move() { return ss->sp->movePicker->next_move(); } +Move MovePicker::next_move() { return ss->splitPoint->movePicker->next_move(); }