summary |
shortlog |
log |
commit | commitdiff |
tree
raw |
patch |
inline | side by side (from parent 1:
bbd3e30)
Creating an History object requires clearing the History tables,
although fast is an useless job in san.cpp where History is used
just as a dummy argument for MovePicker c'tor.
So use a file scoped constant instead of creating a new History()
object each time MovePicker c'tor is called as in move_ambiguity()
This optimizes pretty_pv() through the following calling chain:
pretty_pv() -> line_to_san() -> move_to_san() -> move_ambiguity()
No functional change.
Signed-off-by: Marco Costalba <mcostalba@gmail.com>
enum Ambiguity {
AMBIGUITY_NONE,
AMBIGUITY_FILE,
enum Ambiguity {
AMBIGUITY_NONE,
AMBIGUITY_FILE,
+ const History H; // used as dummy argument for MovePicker c'tor
Ambiguity move_ambiguity(const Position& pos, Move m);
const string time_string(int milliseconds);
Ambiguity move_ambiguity(const Position& pos, Move m);
const string time_string(int milliseconds);
- MovePicker mp = MovePicker(pos, false, MOVE_NONE, OnePly, History());
+ MovePicker mp = MovePicker(pos, false, MOVE_NONE, OnePly, H);
// Castling moves
if (movestr == "O-O-O" || movestr == "O-O-O+")
// Castling moves
if (movestr == "O-O-O" || movestr == "O-O-O+")
if (type_of_piece(pc) == KING)
return AMBIGUITY_NONE;
if (type_of_piece(pc) == KING)
return AMBIGUITY_NONE;
- MovePicker mp = MovePicker(pos, false, MOVE_NONE, OnePly, History());
+ MovePicker mp = MovePicker(pos, false, MOVE_NONE, OnePly, H);
Move mv, moveList[8];
int n = 0;
Move mv, moveList[8];
int n = 0;