kate/BadFish
1
0
Fork 0
mirror of https://github.com/sockspls/badfish synced 2025-04-29 16:23:09 +00:00
Code Issues Releases Wiki Activity

MovePicker: simplify move swapping

Browse source 
We don't need a full swap here because once found
and returned the best move will not be used again.

Signed-off-by: Marco Costalba <mcostalba@gmail.com>
This commit is contained in:
Marco Costalba 2008-09-18 09:51:24 +01:00
parent 5080e72ea5
commit 52487c4f04
1 changed files with 8 additions and 17 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

25
src/movepick.cpp
View file

@ -335,10 +335,8 @@ Move MovePicker::pick_move_from_list() {
}
}
if(bestIndex != -1) { // Found a good capture
MoveStack tmp = moves[movesPicked];
moves[movesPicked] = moves[bestIndex];
moves[bestIndex] = tmp;
move = moves[movesPicked++].move;
move = moves[bestIndex].move;
moves[bestIndex] = moves[movesPicked++];
if(move != ttMove && move != mateKiller &&
pos->move_is_legal(move, pinned))
return move;
@ -368,10 +366,8 @@ Move MovePicker::pick_move_from_list() {
bestIndex = movesPicked;
if(bestIndex != -1) {
MoveStack tmp = moves[movesPicked];
moves[movesPicked] = moves[bestIndex];
moves[bestIndex] = tmp;
move = moves[movesPicked++].move;
move = moves[bestIndex].move;
moves[bestIndex] = moves[movesPicked++];
if(move != ttMove && move != mateKiller &&
pos->move_is_legal(move, pinned))
return move;
@ -392,10 +388,8 @@ Move MovePicker::pick_move_from_list() {
}
if(bestIndex != -1) {
MoveStack tmp = moves[movesPicked];
moves[movesPicked] = moves[bestIndex];
moves[bestIndex] = tmp;
move = moves[movesPicked++].move;
move = moves[bestIndex].move;
moves[bestIndex] = moves[movesPicked++];
return move;
}
}
@ -431,11 +425,8 @@ Move MovePicker::pick_move_from_list() {
bestIndex = movesPicked;
if(bestIndex != -1) {
MoveStack tmp = moves[movesPicked];
moves[movesPicked] = moves[bestIndex];
moves[bestIndex] = tmp;
move = moves[movesPicked++].move;
move = moves[bestIndex].move;
moves[bestIndex] = moves[movesPicked++];
// Remember to change the line below if we decide to hash the qsearch!
// Maybe also postpone the legality check until after futility pruning?
if(/* move != ttMove && */ pos->move_is_legal(move, pinned))